[Segment] Icon with `line` class name breaks layout

[layershifter]

Bug Report

Comes from Semantic-Org/Semantic-UI-React#3888.

Steps to reproduce

1. Use ui segment placeholder and an icon that contains line class name.

Testcase

https://jsfiddle.net/kdjs0awe/2/

Screenshot (if possible)

Version

2.8.6

[layershifter]

It happens because an icon receives unwanted CSS:

[lubber-de]

As suggested in #1620: To fix this we should add 2 one-word alias names for line graph icon and chart line icon : linegraph and chartline to avoid many selector changes from .line to .line:not(.icon)

[lubber-de]

As suggested above, let's just change line graph icon to linegraph icon and chart line icon to chartline icon now.
@ko2in As you assigned this to yourself, would you want to do this little change in icon.overrides ? If yes, then please also create an appropriate PR in the https://github.com/fomantic/create-fomantic-icons repository, so the next time FA icons are created we still have this fixed.
If you don't have time, just tell me (or assign the issue to me) , i am doing it then 🙂
@hammy2899 Last chance in case you don't agree 😆

[ko2in]

@lubber-de I'll look into this tomorrow and send PR.

[y0hami]

@lubber-de @ko2in ye changing icon names is the best approach here IMO