Skip to content

Fix header overflow, fix more apps menu, fix #11552 - #11553

Merged
MorrisJobke merged 1 commit into
masterfrom
header-overflow
Oct 2, 2018
Merged

Fix header overflow, fix more apps menu, fix #11552#11553
MorrisJobke merged 1 commit into
masterfrom
header-overflow

Conversation

@jancborchardt

Copy link
Copy Markdown
Member

Fix#11552, please review @nextcloud/designers :)

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@weeman1337weeman1337 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The more apps menu is working with these changes ✓

@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 2, 2018
@jancborchardt

Copy link
Copy Markdown
MemberAuthor

Backport at #11558 because the breaking change was also backported. 😁

@MorrisJobke
MorrisJobke merged commit 34751e0 into masterOct 2, 2018
@MorrisJobke
MorrisJobke deleted the header-overflow branch October 2, 2018 19:30
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugdesignDesign, UI, UX, etc.regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jancborchardt@MorrisJobke@weeman1337