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

Select & MultiSelect Chip Colors - ROWY-135 #1151

Merged
merged 24 commits into from
Mar 23, 2023

Conversation

devsgnr
Copy link
Contributor

@devsgnr devsgnr commented Mar 8, 2023

closes #514


Screenshot 2023-03-06 at 02 17 47

🚣🏽 Worked on creating color selection menu for Select & MultiSelect chips (12 predefined colors created from the existing color palette and using paletteToMui function to generate the light and dark variants)
🚣🏽 Worked on adding the react-color-palette component to be able to customize the light and dark variant of any colors (predefined or can start from aGray)
🚣🏽 Worked on adding the colors object to the config object in the table's schema
🚣🏽 Made color changes to everywhere that has a chip

Few things that I'm still working on

  • Working on deleting both options and colors values at the same time (blocker)
  • Retain Yes, No, Maybe coloring to live alongside the new coloring system (nice to have)

@devsgnr devsgnr added enhancement New feature or request good first issue Good for newcomers labels Mar 8, 2023
@devsgnr devsgnr self-assigned this Mar 8, 2023
@vercel
Copy link

vercel bot commented Mar 8, 2023

@devsgnr is attempting to deploy a commit to the Rowy Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Mar 8, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
rowy-os ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 17, 2023 at 9:36AM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
rowy-typedoc ⬜️ Ignored (Inspect) Visit Preview Mar 17, 2023 at 9:36AM (UTC)

@devsgnr devsgnr changed the title Rowy 135 Select & MultiSelect Chip Colors - ROWY-135 Mar 8, 2023
@devsgnr
Copy link
Contributor Author

devsgnr commented Mar 8, 2023

For some reason; onChange prop function doesn't run at the same time, or prevents one of the two from running. I tried delaying the running of the function by using async/await and also callbacks but nothing seems to work. Code section below:

https://github.com/devsgnr/rowy/blob/5e9b211ff99b62fdc3ccfcff548e0f9b072fe709/src/components/fields/SingleSelect/Settings.tsx#L59-L72

cc: @shamsmosowi

@devsgnr devsgnr marked this pull request as ready for review March 15, 2023 10:05
@devsgnr devsgnr removed the request for review from harinij March 15, 2023 10:05
setSelectedFields(fieldKeys)
fieldKeys.forEach(field => {
setSelectedFields(fieldKeys);
fieldKeys.forEach((field) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code formatter is making changes to files not relevant to the PR

@shamsmosowi shamsmosowi merged commit 097ab23 into rowyio:develop Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ROWY-135: Select and Multiselect Fields need color selection for fields
3 participants