-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add more colors in dropdown section #1
Comments
I would like to work on this if it's a part of Hacktoberfest. |
Yes @Ayushi7456 please go ahead! 👍 |
Hi, could you please assign me this for hacktoberfest? Also, could you add the hacktober label for it to qualify for hacktoberfest? Thank you |
Yes @Vidushi-Gupta please go ahead 👍 . Add any one color👍 |
Can I add a color |
Can I work on this ? |
Just added a PR. Thank you! |
raised a PR for Linen |
Can I work on this? |
Added 'Maroon' color in the dropdown section
will this pull request count towards hacktober fest |
Hey @vishal-codes I have raised a PR #30 can you give it a hacktoberfest label |
No, at that time I wasn't able to add the labels, Now the labels are available if u make a new pr with new changes then I can give ur pr a label 🚀 |
yaa sure |
Can I do one colour please? |
Go ahead |
Hi! May I add a color too? |
Sure, Go ahead @ayangupta9 |
Hi @vishal-codes Please merge the PR. Thank You |
Hi @vishal-codes I have added DarkOrange and DarkOliveGreen color please merge my PR #58 . Thank you |
Can i work on this issue? |
Added IndianRed and LightBlue colours :D please merge my PR #64 |
I closed this one, cause a refactor was made with how colors options are defined, and on this refactor the color I originally addeed was added too. So, I added another one :) |
I would like to contribute to this issue |
4 colors are added #119 |
Hi @vishal-codes I have added 2 Colors and 2 languages #118 Please Look into it |
Lovely people, it will be a little easy for Maintainers if you do not link this issue! It is getting close whenever we are merging a PR. ヽ(^o^)丿 |
hey, one of my prs have already been merged (which added 2 languages). Can I still contribute on this issue? |
Yeah, why not |
Hi, @vishal-codes i would like to add MediumBlue |
Hey @sadathshariff, you can open a PR. no worries |
@vishal-codes Hey!! I have added two colors plzz merge the PR. #148 |
As of now the Dropdown section of the project only supports two colors (Blue and Yellow). We can add more colours.
Before making changes please make sure that you have completed steps till Branching.
Steps for adding more colours
src/data/colorOptions.js
file.label
andvalue
with the colour name (please follow the casing)Note: Add the colour's in alphabetical order and if u want to get you
PR eligible for Hacktoberfest
then u gotta at least add 2 colours and 2 languages in a single PRList of colour's to be added (NOTE: You don't need to add all the colours, just add one or two colour's. Not more than two.)
The text was updated successfully, but these errors were encountered: