dir="auto" is changed by Dir directive

[alirezamirian]

Bug:

Dir directive treats anything other than rtl and ltr as invalid value, and changes dir attribute to ltr which breaks dir="auto".

What is the expected behavior?

It should not change dir attribute. dir="auto" is perfectly fine.

What is the current behavior?

It changes it to ltr

Which versions of Angular, Material, OS, TypeScript, browsers are affected?

All versions since this pull request is merged.

[alirezamirian]

This fix can be used as a temporary workaround:

import {Directive, HostBinding, Input} from '@angular/core';

/**
 * This is fix for this issue in angular material: https://github.com/angular/material2/issues/13855
 * TODO: remove it when the issue is fixed
 */
@Directive({
  selector: '[dir]'
})
export class MaterialDirFixDirective {

  @HostBinding('attr.dir')
  @Input()
  dir: string;
}

[crisbeto]

We probably shouldn't be overriding the attribute's value, but note that auto isn't supported in any of the Material components anyway.

[alirezamirian]

Yes, I think dir **attribute** should be leaved untouched. Btw, `Dir` directive selector is not specific to material components. I'm actually using `dir="auto"` on an input.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}