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

User management layout improvements #1777

Merged
merged 2 commits into from
Oct 18, 2016
Merged

User management layout improvements #1777

merged 2 commits into from
Oct 18, 2016

Conversation

jancborchardt
Copy link
Member

  • fix list height and border to be as in Files
  • improve look of Add group flow

Before
capture du 2016-10-18 12-18-09

After
capture du 2016-10-18 12-17-10

Much nicer look, better whitespace. A long overdue improvement here ;) Please review @nextcloud/designers

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
@jancborchardt jancborchardt added this to the Nextcloud 11.0 milestone Oct 18, 2016
@mention-bot
Copy link

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

@skjnldsv
Copy link
Member

👍
Much clearer!

@Espina2
Copy link
Contributor

Espina2 commented Oct 18, 2016

Nice improvement.

@MorrisJobke
Copy link
Member

Tested and works 👍

@MorrisJobke
Copy link
Member

CI job is pending because I recently played around with workers and this seem to have failed here.

@MorrisJobke MorrisJobke merged commit 4fbffe0 into master Oct 18, 2016
@MorrisJobke MorrisJobke deleted the user-mgmt-layout branch October 18, 2016 12:57
@MariusBluem
Copy link
Member

I'm lovin it 🍟👍🏼👍🏼

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

Successfully merging this pull request may close these issues.

6 participants