Uh oh!
There was an error while loading. Please reload this page.
Apply border to primary button if on log in page (and not in a dark container) or if in header - #12558
Conversation
…ontainer) or if in header, fix#12294 Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
I'll suggest a translucent border ( |
jancborchardt
commented
Nov 21, 2018
@skjnldsv could you add a commit to implement that, or give me a pointer for this? |
skjnldsv
commented
Nov 21, 2018
@jancborchardt sure, you need to do it on the theming scss. :) |
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
@skjnldsv done, I think. :) Thanks for the pointer! |
jancborchardt
commented
Nov 21, 2018
@schiessle@MariusBluem@juliushaertl@MorrisJobke as you were involved in the "blurry buttons" issue, your review is very welcome! :) |
MorrisJobke
left a comment
There was a problem hiding this comment.
Tested and looks a lot better than before 👍


fix#12294, please review @nextcloud/designers @schiessle
For buttons on dark background, like in the header of the share page, there still is white border:

For buttons elsewhere in the main content, where they are on white background, the border color is the same as the background color:

We could also go with

border:none;which looks even better, but then also need an exception for theming:In general we still need something for light theming colors … as usual. @juliushaertl@skjnldsv any idea? Maybe again cap the color at a specific lightness and then use the standard border color?
I am not sure whether this is something for 15 in any case as this has a bunch of potential for regressions and looking strange too.