Skip to content

Conversation

@juliusknorr
Copy link
Member

2nd part of #770 split

  • Add preview of login screen
  • Hide undo buttons on default values

requires #1026

@juliusknorr
Copy link
Member Author

CI is happy, ready for review @nextcloud/designers @nextcloud/theming

I've included #1026 for easier testing, will rebase to master once this is merged.

@MariusBluem
Copy link
Member

👍

@rullzer
Copy link
Member

rullzer commented Aug 26, 2016

#1026 is in

@juliusknorr juliusknorr force-pushed the theming-admin-improvements branch from 75afc96 to 3d38cb9 Compare August 26, 2016 10:08
@juliusknorr
Copy link
Member Author

@rullzer rebase done.

@nickvergessen
Copy link
Member

Works 👍

@nickvergessen nickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 26, 2016
@MariusBluem MariusBluem merged commit 5c73596 into master Aug 26, 2016
@MariusBluem MariusBluem deleted the theming-admin-improvements branch August 26, 2016 11:37
@MariusBluem
Copy link
Member

Backport to stable10 would be nice 😁 .. So that we have some more theming improvements up there :) @karlitschek

@nickvergessen
Copy link
Member

Or we give people a reason to update to 11?
The fixes in this branch are not really critical but more like enhancements.

@LukasReschke
Copy link
Member

Agreeing with @nickvergessen on that one.

@jancborchardt
Copy link
Member

Yep, let’s not backport everything please. ;)

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

Labels

4. to release Ready to be released and/or waiting for tests to finish enhancement feature: theming

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants