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

Popover to css guidelines - #205

Merged
oparoz merged 2 commits into
masterfrom
popover-to-css-guidelines
Feb 13, 2017
Merged

Popover to css guidelines#205
oparoz merged 2 commits into
masterfrom
popover-to-css-guidelines

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Feb 12, 2017

Copy link
Copy Markdown
Member

capture d ecran_2017-02-12_05-42-18

Refs: nextcloud/server#3118

Ps: non related to this pr, but the other icons are still using img. It could be great if you could use span with icon inside instead (see buttons in files for example)

@skjnldsvskjnldsv added 2. developing Work in progress design Related to the design labels Feb 12, 2017
@skjnldsvskjnldsv self-assigned this Feb 12, 2017
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv
skjnldsvforce-pushed the popover-to-css-guidelines branch from 6817339 to 6020d3dCompareFebruary 12, 2017 04:36
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 12, 2017

@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 👍

@oparoz
oparoz merged commit 26b0e3c into masterFeb 13, 2017
@oparoz
oparoz deleted the popover-to-css-guidelines branch February 13, 2017 11:11
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

3. to reviewWaiting for reviewsdesignRelated to the design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@MorrisJobke@oparoz