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

Revise color selection for calendars #344

Closed
TecJon opened this issue Mar 19, 2016 · 26 comments
Closed

Revise color selection for calendars #344

TecJon opened this issue Mar 19, 2016 · 26 comments

Comments

@TecJon
Copy link

TecJon commented Mar 19, 2016

Some (design) suggestions for the calendar colors

  • Provide more colorful and high-contrast default colors (currently eight).
    • I'd suggest red, orange, yellow, green, blue, purple, pink, brown (similar to OSX)
  • Provide a 'other' or 'more' option to choose a custom color: Richer Calendar colors options #135
    • The 'other' or 'more' could be realized by a plus symbol occupying and replacing the last rectangle to choose a color.
  • Prevent a messy line break when editing a calendar:
    • I'd suggest a single line for colors and a single line for both buttons.

calendarcolor

Here a quick suggestion how it could look like
calendarsuggestion
Note, of course the text input, the colors and the buttons should be aligned on the right.

@georgehrke
Copy link
Contributor

generally speaking create one ticket per actual issue please.

I'm very happy with the current colors and I think the ones you are suggesting are bad for readability.
As you already said there is a ticket for the second issue.

Can we agree to make this ticket about the third issue?

@georgehrke georgehrke added this to the 1.1-current milestone Mar 19, 2016
@georgehrke
Copy link
Contributor

cc @raghunayyar

@TecJon
Copy link
Author

TecJon commented Mar 19, 2016

I wanted this thread to collect ideas. Once agreed, each issue can be opened in a single ticket.

I guess the colors truly are a matter of taste. However, I think current colors 1, 2, and 4 are too similar and including 3 a little too dull. The ones I suggested in the image were quickly chosen, a designer should advise on actual good choices for readability (the red and orange are quite similar, too). Nonetheless, I would vote for more colorful default colors. :-)

@georgehrke
Copy link
Contributor

I would agree that 1,2 and 4 are too similar.
"Dull" colors are not necessarily a bad thing. These colors don't need to be bright and shiny. What's important is that you can read the text very easily. A bright and shiny background is often bad in this regard.

@TecJon
Copy link
Author

TecJon commented Mar 19, 2016

I agree, readability should be the primary concern here.

But I believe readability and a little colorfulness are not mutually exclusive. For example the labels on here on github are quite colorful yet easy to read.
OSX solved the issue by using a lighter version of the color as the background color of the event.

Image of OSX calendar 1

One could think about matching the owncloud contacts app colors.

@georgehrke
Copy link
Contributor

Well, that ain't OS X Calendar but Fantastical, but I like that as well. 😉

@TecJon
Copy link
Author

TecJon commented Mar 19, 2016

Oh haha! Well that's a little unfortunate. 😃

For the sake of completeness
Really OS X calendar

@georgehrke
Copy link
Contributor

cc @jancborchardt

@j-ed
Copy link

j-ed commented Mar 21, 2016

I like the idea of having more colorful and high-contrast default colors too. I cannot understand why a user should be limited to low-contrast default colors only. Many people cannot differenciate colors correctly or need to wear glasses so that it is essential for them to select colors based on their requirements. Unfortunately this is ignored by many developers because they didn't know or due to design descisions. The original color-picker mockup of @SamEliotFisher therefore seems to be the right way to go.

@eesko
Copy link

eesko commented Mar 21, 2016

Custom color selection would be useful. I have a number of calendars that are color coded across all different programs/devices that I use. The limited color choices in the ownCloud calendar app make it impossible to use the colors I use across all other programs/devices. Also if I happen to have more than eight calendars I must currently assign same color for more than one calendar.

I also think that the dark colors are too close to each other. Some of the other default color choices seem odd to me as well. There is a gradient from yellow to orange to red but only one shade of blue and no green at all.

@koenlek
Copy link

koenlek commented Apr 13, 2016

I agree with @eesko, a custom color picker would be great, 8 colors is not enough for me...

@raghunayyar
Copy link
Member

@koenlek @eesko Contributions are welcome. :)

@georgehrke
Copy link
Contributor

As mentioned in this issues description: There is an issue for a rich color picker: #135

@jancborchardt
Copy link
Member

@eesko @koenlek can you elaborate on why this is not enough? How many Calendars do you have?

cc @skjnldsv by the way – it would be good to generate the calendar colors with the core script. Currently they are hard coded.

@skjnldsv
Copy link
Contributor

It's possible! Do you want me to open a pr? :)

@jancborchardt
Copy link
Member

@skjnldsv that would be pretty damn cool! :) Cause the calendar colors are something I overlooked on the list of where we want to stay consistent on colors.

@skjnldsv
Copy link
Contributor

skjnldsv commented Apr 20, 2016

@jancborchardt damn I almost forgot about this issue! :)
Should we use the same colors from owncloud-archive/mail#1434 ?

Or should we add a color wheel?
And the ability to choose a random color from the generator?

@jancborchardt
Copy link
Member

Use the same colors as in Mail I'd say. Not a wheel or generator, just the grid of 8 colors as we do now. Since there's 9 in mail, just cut the turqoise right next to green – not that easy to distinguish anyway.

@skjnldsv
Copy link
Contributor

That's a shame I can't choose my own color though! :/

@jancborchardt
Copy link
Member

Well, first comes the proper color adjustment. Custom colors is a separate thing and has way lower priority. ;)

@jancborchardt
Copy link
Member

(Way lower priority than some other Calendar issues, that is. :)

@skjnldsv
Copy link
Contributor

:p

@skjnldsv
Copy link
Contributor

@jancborchardt Hum, I've been thinking, and I don't think we can use the core generator. The calendar app is using a defined color code, and it's not using a string or a hash to generate a color. Aside from using the idea of associating a hash to a calendar and use it for the color generator, I can't use the color function to generate a set of valid color for the user to choose.

I don't see if i'm understandable or not, but this should be an issue.
I really think we should associate a calendar id and let the user change it to generate a new colour for his calendar.

@skjnldsv
Copy link
Contributor

skjnldsv commented May 1, 2016

@jancborchardt bump

@jancborchardt
Copy link
Member

@skjnldsv right – what we can do easiest at the moment is to simply get 8 colors out of the algorithm across the color spectrum and replace the hardcoded list in calendar with those. That way calendar will use the same color spectrum as the rest. Ok?

@skjnldsv
Copy link
Contributor

skjnldsv commented May 2, 2016

Ok, will work on it this week! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants