New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefix icon variables with app name to avoid conflicts #12421

Merged
merged 1 commit into from Nov 15, 2018

Conversation

Projects
None yet
4 participants
@juliushaertl
Copy link
Member

juliushaertl commented Nov 12, 2018

Otherwise apps with the same icon name will overwrite each other.

Example:

deck uses: @include icon-color('app', 'deck', $color-black);
calendar uses: @include icon-color('app', 'calendar', $color-black);

Before:

Both icon variables have the same name --icon-app-000

Before:

deck uses --icon-deck-app-000
calendar uses --icon-calendar-app-000

@skjnldsv
Copy link
Member

skjnldsv left a comment

Awesome stuff on your last pr!! You 🎸 !!

@MorrisJobke

This comment has been minimized.

Copy link
Member

MorrisJobke commented Nov 12, 2018

Jsunit tests failed 🙈

@JanDragon
Copy link

JanDragon left a comment

Great

@MorrisJobke MorrisJobke referenced this pull request Nov 14, 2018

Merged

15.0.0 Beta 2 #12355

16 of 24 tasks complete
@MorrisJobke

This comment has been minimized.

Copy link
Member

MorrisJobke commented Nov 14, 2018

@juliushaertl Would be nice to get them fixed for tomorrow 😃

@juliushaertl juliushaertl force-pushed the bugfix/noid/icons-cacher-app-prefix branch from a95fc1a to 56ad676 Nov 15, 2018

@MorrisJobke

This comment has been minimized.

Copy link
Member

MorrisJobke commented Nov 15, 2018

JSUnit still fails 🙈

@juliushaertl juliushaertl force-pushed the bugfix/noid/icons-cacher-app-prefix branch from 56ad676 to da6cf15 Nov 15, 2018

Prefix icon variables with app name to avoid conflicts
Signed-off-by: Julius Härtl <jus@bitgrid.net>

@juliushaertl juliushaertl force-pushed the bugfix/noid/icons-cacher-app-prefix branch from da6cf15 to 521b2dd Nov 15, 2018

@juliushaertl

This comment has been minimized.

@MorrisJobke MorrisJobke merged commit 7baa67c into master Nov 15, 2018

1 check failed

continuous-integration/drone/pr the build failed
Details

@MorrisJobke MorrisJobke deleted the bugfix/noid/icons-cacher-app-prefix branch Nov 15, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment