Skip to content

Conversation

@skjnldsv
Copy link
Member

The open class is needed only when opened.
After #3024 the workarround will not work and the design will have to fit the css guidelines :)

@skjnldsv skjnldsv added 3. to review Waiting for reviews design Design, UI, UX, etc. feature: federation low labels Jan 11, 2017
@skjnldsv skjnldsv added this to the Nextcloud 12.0 milestone Jan 11, 2017
@skjnldsv skjnldsv self-assigned this Jan 11, 2017
@mention-bot
Copy link

@skjnldsv, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ChristophWurst, @schiessle and @LukasReschke to be potential reviewers.

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the fix-federationScopeMenu-menu branch from 75abd75 to a3e3111 Compare January 11, 2017 20:20
@skjnldsv
Copy link
Member Author

Also, after #3024, I added the center class to menu. We could probably get rid of the js magic for the popups :)

@MorrisJobke MorrisJobke merged commit 00c3f80 into master Jan 12, 2017
@MorrisJobke MorrisJobke deleted the fix-federationScopeMenu-menu branch January 12, 2017 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews design Design, UI, UX, etc. feature: federation low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants