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

fix: Tokenizer Input + Shellbar button improve specificity #754

Merged
merged 1 commit into from Mar 3, 2020

Conversation

jbadan
Copy link
Contributor

@jbadan jbadan commented Mar 3, 2020

Related Issue

Closes SAP/fundamental-styles#

Description

Specificity of (0,0,0) is not allowing fd-shellbar__button to override fd-button, same with fd-tokenizer__input and fd-input. These !important allow these to override. This is not seen on the fundamental styles docs site due to the order in the fundamental-styles.css file.

In Fundamental-react:
Screen Shot 2020-03-03 at 12 37 27 PM
Screen Shot 2020-03-03 at 12 38 06 PM

@jbadan jbadan requested review from greg-a-smith and a team March 3, 2020 20:39
@netlify
Copy link

netlify bot commented Mar 3, 2020

Deploy preview for fundamental-styles ready!

Built with commit e68181b

https://deploy-preview-754--fundamental-styles.netlify.com

@jbadan jbadan merged commit aed04cd into master Mar 3, 2020
@jbadan jbadan deleted the fix/specificity branch March 3, 2020 21:08
@mikerodonnell89
Copy link
Member

I don't think using !important is improving specificity, it's just forcing an override. I think this would be a better solution (in the example of the shellbar) #744

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

Successfully merging this pull request may close these issues.

None yet

3 participants