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

namespace icon classes to avoid style conflict #5

Closed
simpixelated opened this issue Mar 9, 2016 · 4 comments
Closed

namespace icon classes to avoid style conflict #5

simpixelated opened this issue Mar 9, 2016 · 4 comments

Comments

@simpixelated
Copy link
Contributor

When using Twitter Bootstrap, I noticed one conflict with .blockquote, which breaks the button layout:
screen shot 2016-03-09 at 2 56 25 pm

My suggestion is to namespace all of the icon classes, maybe with just icon- (e.g. icon-blockquote).

@KyleAMathews
Copy link

Or perhaps use CSS-modules
On Wed, Mar 9, 2016 at 2:59 PM Jordan Kohl notifications@github.com wrote:

When using Twitter Bootstrap, I noticed one conflict with .blockquote
http://v4-alpha.getbootstrap.com/content/reboot/#blockquote, which
breaks the button layout:
[image: screen shot 2016-03-09 at 2 56 25 pm]
https://cloud.githubusercontent.com/assets/838282/13653539/2330636a-e607-11e5-9199-539e5ffa746c.png

My suggestion is to namespace all of the icon classes, maybe with just
icon- (e.g. icon-bootstrap).


Reply to this email directly or view it on GitHub
#5.

@simpixelated
Copy link
Contributor Author

@KyleAMathews agreed; I just don't know enough about CSS-modules to make a pull request -- but will look into it. Can it be added as a loader/build step with just Webpack?

@sstur sstur closed this as completed in #7 Mar 9, 2016
@sstur
Copy link
Owner

sstur commented Mar 10, 2016

Thanks @KyleAMathews. I am planning to move to CSS Modules soon (It's in the TODO in the readme). Will have created a separate issue to track that progress: #10

@KyleAMathews
Copy link

👍
On Wed, Mar 9, 2016 at 6:13 PM Simon Sturmer notifications@github.com
wrote:

Thanks @KyleAMathews https://github.com/KyleAMathews. I am planning to
move to CSS Modules soon (It's in the TODO in the readme). Will have
created a separate issue to track that progress: #10
#10


Reply to this email directly or view it on GitHub
#5 (comment).

ljorgens pushed a commit to Pick6Solutions/react-rte that referenced this issue Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants