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

More useful colors, Bootstrap convention and compatibility #126

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

More useful colors, Bootstrap convention and compatibility #126

wants to merge 3 commits into from

Conversation

majnooni
Copy link

Hi.

I am very happy to use this repo during recent months and I really thank you for that. I usually tend to use CDN links but at the same time I like to have more options in terms of customisation. Therefore, I was wondering if I could be contributing to this repo and then have a better customised CDN. These are my changes to the repo and I hope they will be ok.

  • Beautiful colors are added from "persian" family.

  • Some branding colors (facebook, ...) are added since they are widely seen by the users and present a sense of intimacy. These new colors also bring simplicity in UI design. Blue variation has been point of concern as it is regarded as peaceful, tranquil, secure, and orderly.

  • Since this css will be probably used as a complementary color palette to Bootstrap framework (as I usually do), some css classes have been renamed to be consistent with bootstrap conventions introducing more integrity in coding (.{color} -> .text-{color} & .border--{color} -> .border-{color}).

  • As this css color palette can be used individually, bootstrap colors (primary, secondary , ...) are also introduced providing a wide range of options in hand.

  • "!important" declaration is added to provide specificity.

Feel free to edit in case of need.

Regards
Meysam Majnooni

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.

1 participant