-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
Added alias for euro -> eur #14869
Added alias for euro -> eur #14869
Conversation
Due to backward compatibility, we can't actually rename it. We could add an alias though. |
Currently there are no aliases as far as I know.. Should I just update the .less file? |
Squashed and just added another entry.. let me know if that's fine or we can just forget about this :) |
@lipis: you need to do fetch and rebase for Travis to pass. This seems a right addition and in fact |
@@ -34,6 +34,7 @@ | |||
.glyphicon-asterisk { &:before { content: "\2a"; } } | |||
.glyphicon-plus { &:before { content: "\2b"; } } | |||
.glyphicon-euro { &:before { content: "\20ac"; } } | |||
.glyphicon-eur { &:before { content: "\20ac"; } } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just do it like this?
.glyphicon-euro,
.glyphicon-eur { &:before { content: "\20ac"; } }
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right.. fixed..
@XhmikosR Travis is green now :) |
Fixes #14867
I didn't update the minified/map files but could do that if necessary.. or not update some of the files that I updated already :)