Skip to content

Conversation

@NishchintDhawan
Copy link

@NishchintDhawan NishchintDhawan commented Jun 5, 2022

Description

This PR provides the user to select a custom color for the courses currently selected. The current implementation provides a way to select a color from the provided list of colors which comes up when the user clicks on the edit icon.

Closes #361

Screenshots

After

When the user has clicked on the edit icon:
image

Dark mode:
image

Checklist

  • The code follows all style guidelines.
  • The code passes all required tests.
  • The code is documented.
  • The code includes tests.
  • I have self-reviewed my changes and have done QA.

General Comments

The user interaction is not finalised but just one of the ways to solve the issue. Another option could be to provide custom colors. The issue with that is if the user chooses white or black as colors, the edit button might not be visible along with the room number. Open for discussion !

@NishchintDhawan NishchintDhawan requested a review from a team as a code owner June 5, 2022 05:10
@vercel
Copy link

vercel bot commented Jun 5, 2022

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

Name Status Preview Updated
courseup ✅ Ready (Inspect) Visit Preview Jun 5, 2022 at 5:45AM (UTC)

@NishchintDhawan NishchintDhawan changed the title Nishchint/custom color for courses Custom color for courses Jun 5, 2022
@NishchintDhawan NishchintDhawan marked this pull request as draft June 5, 2022 05:12
@FlyteWizard
Copy link
Member

Another option could be to provide custom colours. The issue with that is if the user chooses white or black as colours, the edit button might not be visible along with the room number.

A possible solution for the future is color-contrast() but this is still a Safari experimental feature.

https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/color-contrast

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.

Allow users to set custom colours for courses

3 participants