Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Fixed too wide selector for the popovermenu #325

Merged
merged 1 commit into from
Nov 10, 2017
Merged

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Nov 10, 2017

This was interfering with some other menus on the nextcloud server

@nextcloud/javascript @nextcloud/designers

@skjnldsv skjnldsv added 3. to review Waiting for reviews bug Something isn't working labels Nov 10, 2017
@skjnldsv skjnldsv self-assigned this Nov 10, 2017
@skjnldsv skjnldsv mentioned this pull request Nov 10, 2017
12 tasks
This was interfering with some other menus on the nextcloud app.

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv
Copy link
Member Author

Test failure not related to this pr. Master is broken apparently! https://travis-ci.org/nextcloud/gallery/builds

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

Seems to do the trick :)

@rullzer rullzer requested a review from oparoz November 10, 2017 18:19
@oparoz oparoz merged commit dbfef58 into master Nov 10, 2017
@oparoz oparoz deleted the popovermenu-selector-fix branch November 10, 2017 18:37
@blizzz
Copy link
Member

blizzz commented Nov 10, 2017

gnah, within vendor/, no wonder ack didn't find that occurence when i was looking for "icon-more"

@skjnldsv
Copy link
Member Author

@blizzz same, my grep exclude was messing with me! 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3. to review Waiting for reviews bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants