Skip to content
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

Merged
merged 1 commit into from
Oct 26, 2014
Merged

Added alias for euro -> eur #14869

merged 1 commit into from
Oct 26, 2014

Conversation

lipis
Copy link
Contributor

@lipis lipis commented Oct 24, 2014

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 :)

@cvrebert
Copy link
Collaborator

Due to backward compatibility, we can't actually rename it. We could add an alias though.

@lipis
Copy link
Contributor Author

lipis commented Oct 24, 2014

Currently there are no aliases as far as I know.. Should I just update the .less file?

@lipis lipis changed the title Renamed glyphicon-euro to glyphicon-eur Added alias for euro -> eur Oct 24, 2014
@lipis
Copy link
Contributor Author

lipis commented Oct 24, 2014

Squashed and just added another entry.. let me know if that's fine or we can just forget about this :)

@XhmikosR XhmikosR added the css label Oct 24, 2014
@XhmikosR
Copy link
Member

@lipis: you need to do fetch and rebase for Travis to pass.

This seems a right addition and in fact euro is inconsistent from the beginning.

@@ -34,6 +34,7 @@
.glyphicon-asterisk { &:before { content: "\2a"; } }
.glyphicon-plus { &:before { content: "\2b"; } }
.glyphicon-euro { &:before { content: "\20ac"; } }
.glyphicon-eur { &:before { content: "\20ac"; } }
Copy link
Collaborator

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"; } }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right.. fixed..

@lipis
Copy link
Contributor Author

lipis commented Oct 24, 2014

@XhmikosR Travis is green now :)

@mdo mdo added this to the v3.3.0 milestone Oct 26, 2014
mdo added a commit that referenced this pull request Oct 26, 2014
@mdo mdo merged commit 57b60e3 into twbs:master Oct 26, 2014
@cvrebert cvrebert mentioned this pull request Oct 26, 2014
@lipis lipis deleted the glyphicon-eur branch October 26, 2014 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

glyphicon-euro incorrectly named
5 participants