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

Own background image conflicts with admin theming primary color #33925

Closed
3 of 4 tasks
jancborchardt opened this issue Sep 6, 2022 · 13 comments
Closed
3 of 4 tasks

Own background image conflicts with admin theming primary color #33925

jancborchardt opened this issue Sep 6, 2022 · 13 comments

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Sep 6, 2022

Problem

With the design update, you can have the strange state that the primary color picked by the admin is still present when you pick a different background image, resulting in e.g. red primary buttons when there is a green-ish background image.

@skjnldsv since we also talked about that in #33810 (comment)

Possible solution

For users
People should be able to set their own design, background image including primary color. For that we can do several enhancements, any or all of these:

For admins

Default
The default image and color if nothing is customized by either admin or individual people remains:

  • background (for logged in as well as logged out pages): kamil-porembinski-clouds.jpg
  • color: Nextcloud blue #0082c9

cc @PVince81 @skjnldsv @karlitschek @AndyScherzinger


List of colors for default dashboard images (taken by putting the images through the Material Theme Builder):

  • #a53c17 anatoly-mikhaltsov-butterfly-wing-scale.jpg
  • #316b26 bernard-spragg-new-zealand-fern.jpg
  • #56633d bernie-cetonia-aurata-take-off-composition.jpg
  • #9c4236 dejan-krsmanovic-ribbed-red-metal.jpg
  • #4f6071 eduardo-neves-pedra-azul.jpg
  • #396475 european-space-agency-barents-bloom.jpg
  • #98415a hannes-fritz-flippity-floppity.jpg
  • #845334 hannes-fritz-roulette.jpg
  • #4f6071 hannes-fritz-sea-spray.jpg
  • #0082c9 kamil-porembinski-clouds.jpg (the actual accessible color here would be #00639a)
  • #7f5700 lali-masriera-yellow-bricks.jpg
  • #005ac1 nasa-waxing-crescent-moon.jpg
  • #7b4e7e rawpixel-pink-tapioca-bubbles.jpg
  • #6a2af4 tommy-chau-already.jpg
  • #7f4f70 tommy-chau-lion-rock-hill.jpg
@Pytal
Copy link
Member

Pytal commented Sep 27, 2022

As a user this would be much simpler and less of a footgun by not having a custom color picker and instead just picking the background which automatically chooses the primary color resulting in a background and color combination that never conflict, what do you think @jancborchardt?

Color presets added in #34272

@szaimen
Copy link
Contributor

szaimen commented Sep 27, 2022

As a user this would be much simpler and less of a footgun by not having a custom color picker and instead just picking the background which automatically chooses the primary color resulting in a background and color combination that never conflict, what do you think @jancborchardt?

Color presets added in #34272

Makes sense! But what abour custom backgrounds?

@szaimen
Copy link
Contributor

szaimen commented Sep 27, 2022

ping @jancborchardt

@jancborchardt
Copy link
Member Author

Yes, the default presets for the default backgrounds are good, thanks @Pytal! :)

As mentioned in the post, it would still be great if a color picker shows up when you load a custom image (does not need to show for the default pictures as the colors are good).

And in the future, we can get the color automatically from custom images too – or if it's easier we do that directly. Then no footgun there too. ;)

@Pytal
Copy link
Member

Pytal commented Sep 28, 2022

New color presets used in #34298

@skjnldsv skjnldsv added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Sep 28, 2022
@nickvergessen
Copy link
Member

Please note/adjust tertiary-on-primary in the vue library accordingly as well.
The name might not be matching anymore but it is only used for buttons that are added in the top bar, e.g. in Talk to open the sidebar.

@Tealk
Copy link

Tealk commented Oct 10, 2022

Does the following problem also apply here?

A separate background, the upper edge is bright
Theme Color #078D4E

Unfortunately I don't know how to get the navigation dark like it is with some of the preinstalled background images.
image

@Pytal
Copy link
Member

Pytal commented Oct 11, 2022

Rewrite of admin theming in #34359

@Pytal
Copy link
Member

Pytal commented Oct 12, 2022

Using dark navigation icons for custom backgrounds would be a separate issue @Tealk, feel free to open a new issue

@PVince81
Copy link
Member

@PVince81
Copy link
Member

PVince81 commented Oct 14, 2022

@skjnldsv skjnldsv self-assigned this Oct 14, 2022
@PVince81
Copy link
Member

PVince81 commented Oct 14, 2022

@skjnldsv
Copy link
Member

Fixed now I guess with the various PR and the last #34576
Closing as there is individual tickets for the extra points

Repository owner moved this from 🏗️ In progress to ☑️ Done in 🖍 Design team Oct 21, 2022
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

7 participants