Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

Add RTL support for directional MDL2 icons #792

Merged
merged 6 commits into from
Sep 8, 2016
Merged

Conversation

leddie24
Copy link
Contributor

@leddie24 leddie24 commented Sep 1, 2016

Updated icon-page with icons in Fabric.Icons.Output.scss that were missing.
Updated Fabric.Icons.RTL.Output with new MDL2 directional icons.

…c.Icons.RTL.Output with new MDL2 directional icons
.ms-Icon--RevToggleKey,
.ms-Icon--RightDoubleQuote,
.ms-Icon--CloudWeather,
.ms-Icon--CLoudy,
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like a typo here.

@mikewheaton
Copy link
Contributor

mikewheaton commented Sep 2, 2016

Approved! This looks great 👊

Approved with PullApprove

@mikewheaton
Copy link
Contributor

We do have one icon that can't be mirrored, as the content is different between LTR and RTL:

image

Those icons are named BidiLtr and BidiRtl. We'll need to swap the content something like this:

.ms-Icon--BidiLtr::before { @include ms-Icon--BidiRtl }
.ms-Icon--BidiRtl::before { @include ms-Icon--BidiLtr }

@mikewheaton
Copy link
Contributor

Approved, thanks!

@philworthington
Copy link
Contributor

philworthington commented Sep 8, 2016

Approved

Approved with PullApprove

@leddie24 leddie24 merged commit 180ac15 into master Sep 8, 2016
@leddie24 leddie24 deleted the leddie24/rtl-output-fix branch September 8, 2016 18:22
@mikewheaton mikewheaton added this to the 4.1.0 milestone Sep 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants