Skip to content

fix(bidi): handle uppercase values correctly - #14773

Merged
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:dir-case-insensitive
Feb 14, 2019
Merged

fix(bidi): handle uppercase values correctly#14773
jelbourn merged 1 commit into
angular:masterfrom
crisbeto:dir-case-insensitive

Conversation

@crisbeto

Copy link
Copy Markdown
Member

The native dir attribute is case-insensitive, however our dir directive will normalize something like RTL to ltr. These changes account for different cases of the values.

@crisbetocrisbeto added the target: patch This PR is targeted for the next patch release label Jan 9, 2019
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jan 9, 2019

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jelbournjelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Jan 9, 2019
The native `dir` attribute is case-insensitive, however our `dir` directive will normalize something like `RTL` to `ltr`. These changes account for different cases of the values.
@crisbeto
crisbetoforce-pushed the dir-case-insensitive branch from 94c171e to 18d3bc0CompareFebruary 1, 2019 21:28
@jelbourn
jelbourn merged commit 6ea0e96 into angular:masterFeb 14, 2019
jelbourn pushed a commit that referenced this pull request Feb 20, 2019
The native `dir` attribute is case-insensitive, however our `dir` directive will normalize something like `RTL` to `ltr`. These changes account for different cases of the values.
@angular-automatic-lock-bot

Copy link
Copy Markdown

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.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: patchThis PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@crisbeto@jelbourn@googlebot