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

Custom color for courses #391

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

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
2 participants