-
Notifications
You must be signed in to change notification settings - Fork 247
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
Allow to set custom color per event #1614
Conversation
This comment has been minimized.
This comment has been minimized.
a118a32
to
98be1f7
Compare
98be1f7
to
4b2360b
Compare
…or.js Signed-off-by: Georg Ehrke <developer@georgehrke.com>
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
4b2360b
to
714ece4
Compare
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
714ece4
to
ccfa924
Compare
@tcitworld Should we just hide the color in read-only mode? |
Not sure the themed illustration pic would be enough, so I think we should keep it for now. Btw, the "x button" to reset the color is hard to find at the end of the line (and low contrast). |
I think it makes more sense if it's a "reset" icon next to it. |
How would it look with the color input being the full width? Then everything would be aligned. |
We could display the CSS color name aside the input since we have it (joking, don't do that). What we have in #1614 (comment) seems fine to me. |
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
I changed it to #1614 (comment) |
@tcitworld Good to go? :) |
When I change the color, it looks good on the day and week calendar views, but appears merely as a small colored dot on the month view. Being slightly color blind, the small dot is easy to miss. Can we change the background color on the month view, like we do on the day and week views? |
@MartinConsultingServicesInc use the forums please https://help.nextcloud.com/c/apps/calendar/42 |
fixes #71