Skip to content

Fix apps menu on mobile - #10853

Merged
juliusknorr merged 2 commits into
masterfrom
apps-menu-fix-mobile
Aug 30, 2018
Merged

Fix apps menu on mobile#10853
juliusknorr merged 2 commits into
masterfrom
apps-menu-fix-mobile

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added design Design, UI, UX, etc. 3. to review Waiting for reviews regression labels Aug 25, 2018
@skjnldsvskjnldsv added this to the Nextcloud 14 milestone Aug 25, 2018
@skjnldsvskjnldsv self-assigned this Aug 25, 2018
juliusknorr
juliusknorr previously requested changes Aug 25, 2018

@juliusknorrjuliusknorr 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.

@skjnldsv This breaks the other popovers on mobile:

image

@rullzer

Copy link
Copy Markdown
Member

@skjnldsv see the comment from @juliushaertl

@MorrisJobkeMorrisJobke mentioned this pull request Aug 26, 2018
6 tasks
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

@juliushaertl updated and fixed the typo!

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

@juliushaertl 🏓

@juliusknorr
juliusknorr merged commit cdc4eee into masterAug 30, 2018
@juliusknorr
juliusknorr deleted the apps-menu-fix-mobile branch August 30, 2018 12:51
@skjnldsvskjnldsv mentioned this pull request Sep 26, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsdesignDesign, UI, UX, etc.regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@rullzer@juliusknorr