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

"link" icon is displayed instead of the correct icon #6180

Closed
Baloche opened this issue Feb 20, 2018 · 3 comments
Closed

"link" icon is displayed instead of the correct icon #6180

Baloche opened this issue Feb 20, 2018 · 3 comments

Comments

@Baloche
Copy link

Baloche commented Feb 20, 2018

I just updated Semantic UI, and I noticed that for some icons, when I want a link behaviour on my icon, I instead get the "link" icon.

<!-- Here, "link" icon is displayed instead of "retweet" icon -->
<i class="retweet link icon"></i>

I believe this is due to the "link" style definition overriding every previous single-named icon definition.

@Baloche
Copy link
Author

Baloche commented Feb 20, 2018

Either the name of the "link" icon or the name of the "link" behaviour (pointer + opacity change on hover) has to change.

y0hami pushed a commit to y0hami/Semantic-UI that referenced this issue Feb 20, 2018
Fixed arrows.alternate.horizontal
Fixed arrows.alternate.vertical
Fixed arrow.alternate.circle.*
Fixed user.circle.outline Fixes Semantic-Org#6176
Removed link and left linkify icon Fixes Semantic-Org#6180
Bumped envelope unicode to stop conflict with resize.vertical
@y0hami y0hami added this to the 2.3.1 milestone Feb 24, 2018
exoego pushed a commit to exoego/Semantic-UI that referenced this issue Feb 25, 2018
Fixed arrows.alternate.horizontal
Fixed arrows.alternate.vertical
Fixed arrow.alternate.circle.*
Fixed user.circle.outline Fixes Semantic-Org#6176
Removed link and left linkify icon Fixes Semantic-Org#6180
Bumped envelope unicode to stop conflict with resize.vertical
@jlukic
Copy link
Member

jlukic commented Feb 28, 2018

There shouldnt be an icon named link it should be linkify this is no good.

@jlukic
Copy link
Member

jlukic commented Mar 19, 2018

This is fixed in next.

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

No branches or pull requests

3 participants