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

Fix checkboxes when theming color is enabled #2157

Merged
merged 2 commits into from
Nov 16, 2016
Merged

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Nov 16, 2016

fix #2089

Before
2016-11-16-140019_354x186_scrot
After
2016-11-16-140025_353x185_scrot

Checkboxes in user management also still work (#1782) - These should also be tested since the original fix #1971 has been reverted here.

Please review @nextcloud/designers @nickvergessen

This reverts commit bc5268b.

Signed-off-by: Julius Haertl <jus@bitgrid.net>
@juliushaertl juliushaertl added 3. to review Waiting for reviews bug design Design, UI, UX, etc. feature: theming labels Nov 16, 2016
@juliushaertl juliushaertl added this to the Nextcloud 11.0 milestone Nov 16, 2016
@mention-bot
Copy link

@juliushaertl, thanks for your PR! By analyzing the history of the files in this pull request, we identified @skjnldsv, @jancborchardt and @LukasReschke to be potential reviewers.

Signed-off-by: Julius Haertl <jus@bitgrid.net>
@nickvergessen
Copy link
Member

Works, as soon as the css file is rewritten (by changing the color)

👍

@skjnldsv
Copy link
Member

What about the previous issue with the select in dropdown from the users management? Is it still fixed?

@juliushaertl
Copy link
Member Author

@skjnldsv It is. 😉

@skjnldsv
Copy link
Member

skjnldsv commented Nov 16, 2016

👍 🎉

@nickvergessen nickvergessen merged commit 4035d60 into master Nov 16, 2016
@nickvergessen nickvergessen deleted the fix-checkboxes branch November 16, 2016 18:25
@MorrisJobke MorrisJobke mentioned this pull request Nov 16, 2016
67 tasks
@jancborchardt
Copy link
Member

Awesome work @juliushaertl 🤘 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug design Design, UI, UX, etc. feature: theming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Checkbox alignment is broken
5 participants