fix(tooltip): not using trigger's text direction (#4413)

Passes the trigger's layout direction along to the tooltip element. Fixes #4411.
angular · May 17, 2017 · d8aeeaa · d8aeeaa
1 parent ef6b9aa
commit d8aeeaa
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/lib/tooltip/tooltip.spec.ts b/src/lib/tooltip/tooltip.spec.ts
@@ -353,6 +353,19 @@ describe('MdTooltip', () => {
         tooltipDirective.show();
       }).toThrowError('Tooltip position "everywhere" is invalid.');
     });
+
+    it('should pass the layout direction to the tooltip', fakeAsync(() => {
+      dir.value = 'rtl';
+
+      tooltipDirective.show();
+      tick(0);
+      fixture.detectChanges();
+
+      const tooltipWrapper = overlayContainerElement.querySelector('.cdk-overlay-pane');
+
+      expect(tooltipWrapper).toBeTruthy('Expected tooltip to be shown.');
+      expect(tooltipWrapper.getAttribute('dir')).toBe('rtl', 'Expected tooltip to be in RTL mode.');
+    }));
   });
 
   describe('scrollable usage', () => {

diff --git a/src/lib/tooltip/tooltip.ts b/src/lib/tooltip/tooltip.ts
@@ -235,7 +235,9 @@ export class MdTooltip implements OnDestroy {
         this.hide(0);
       }
     });
+
     let config = new OverlayState();
+    config.direction = this._dir ? this._dir.value : 'ltr';
     config.positionStrategy = strategy;
     config.scrollStrategy =
         new RepositionScrollStrategy(this._scrollDispatcher, SCROLL_THROTTLE_MS);