Skip to content

Merge similar properties and standardize the arrow to the menu class - #3282

Merged
MorrisJobke merged 1 commit into
masterfrom
header-menu-arrow-up
Jan 30, 2017
Merged

MorrisJobke merged 1 commit into
masterfrom
header-menu-arrow-up

Conversation

@skjnldsv

@skjnldsv skjnldsv commented Jan 26, 2017 •

Copy link
Copy Markdown
Member
  1. set the menu class
  2. define the position of the :after element

fix ref #3274

capture d ecran_2017-01-26_15-58-48 capture d ecran_2017-01-26_15-58-54 capture d ecran_2017-01-26_15-59-02

@nextcloud/designers

@skjnldsv skjnldsv added 3. to review Waiting for reviews design Design, UI, UX, etc. enhancement labels Jan 26, 2017
@skjnldsv skjnldsv added this to the Nextcloud 12.0 milestone Jan 26, 2017
@skjnldsv skjnldsv self-assigned this Jan 26, 2017
skjnldsv added a commit to nextcloud/notifications that referenced this pull request Jan 26, 2017
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@mention-bot

Copy link
Copy Markdown

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

skjnldsv added a commit to nextcloud/notifications that referenced this pull request Jan 26, 2017
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>

@MorrisJobke MorrisJobke 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 - but doesn't fix #3274 entirely (because the PR from the notification repo is needed as well;))

@skjnldsv

Copy link
Copy Markdown
Member Author

You're right. This is a split pr! :)

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

looks good!

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 30, 2017
@ChristophWurst

Copy link
Copy Markdown
Member

Failing CI run seems unrelated

@skjnldsv

Copy link
Copy Markdown
Member Author

Restarted the test

@MorrisJobke
MorrisJobke merged commit 5618287 into master Jan 30, 2017
@MorrisJobke
MorrisJobke deleted the header-menu-arrow-up branch January 30, 2017 23:23
jancborchardt pushed a commit to nextcloud/notifications that referenced this pull request Mar 27, 2017
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
nickvergessen added a commit to nextcloud/notifications that referenced this pull request Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish design Design, UI, UX, etc. enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants