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

feat: add option to edit the color of a list #1033

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukascorona
Copy link

Description:

This pull request introduces the functionality to edit the background color of a list as mentioned in #840. The feature uses the already existing update functionality, has been implemented across both the server and client and includes the necessary database migration.
Since this would be the third time "color buttons" would be used, I refactored it into a standalone ColorPicker component.

Changelog

  • add a new color column to the list table to store the color. When no color is set this defaults to null.
  • add a reusable ColorPicker component, which is now also used by the ProjectSettingsModal and the LabelEditor
  • introduce a new option in the lists action "edit color" see:
    image
  • Lists can now be set to 10 different colors
  • When setting a color, list header and the button to add a new card switch to white to make it more readable (similar to the text of the label)
    image

Request for Feedback:

I have tried to stick to the existing structure and design patterns as much as possible. I hope it works for you. The documentation is rather scarce and has no section for this, please provide feedback.

As possible colors, I just went with the first ten color options that labels have. I think Lists won't need that many colors, but I can easily add more. Feedback appreciated :)

@CLAassistant
Copy link

CLAassistant commented Feb 18, 2025

CLA assistant check
All committers have signed the CLA.

@VULGA01
Copy link

VULGA01 commented Feb 18, 2025

Hello, how do I install these modifications? I've been waiting for this for months

@VULGA01
Copy link

VULGA01 commented Feb 18, 2025

and is it possible to put a dark grey color like a dark mode ?

@lukascorona
Copy link
Author

Hi @VULGA01

this feature is currently in code review stage and needs to be checked by the maintainer before it can be released.

If you want to test it beforehand, you would need to setup a local development version to run or build the application from scratch by using my branch instead of the master branch. However this is just testing and isn't recommended for any production use.

@meltyshev
Copy link
Member

Hey! Thanks for your pull request. I’m dealing with some health issues right now, but I’ll definitely check everything once I’m better.

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.

4 participants