Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

URL input popover visual cleanup #13973

Merged
merged 6 commits into from
Feb 21, 2019
Merged

URL input popover visual cleanup #13973

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
47acdb2
Use chevron instead of ellipsis in url input field options.
kjellr Feb 19, 2019
a35b7a7
Mimic toolbar icon styles for the icons in the URL popover.
kjellr Feb 19, 2019
6cdf20b
Add a left divider before the URL settings toggle
kjellr Feb 19, 2019
3fb56cd
Even up the spacing in the settings panel.
kjellr Feb 20, 2019
b12a17f
Add periods to code comments.
kjellr Feb 21, 2019
02acf7e
Update snapshot
kjellr Feb 21, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add a left divider before the URL settings toggle
  • Loading branch information
@kjellr
kjellr committed Feb 19, 2019
commit 6cdf20b33bd094f1b6b4dfa4e6513ce9fe7e610b
5 changes: 5 additions & 0 deletions packages/editor/src/components/url-popover/style.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,11 @@
&__settings-toggle {
Copy link
Contributor

Choose a reason for hiding this comment

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

I know you didn't write this initially, but while you're here do you think we should change this rule to be less obtuse? Although there isn't any written rule saying this much, in general I believe the SCSS standard pattern that is emerging is to use write .editor-url-popover__settings-toggle rather than .editor-url-popover { &__settings-toggle { ... } }.

Hmm actually that might be a big rewrite of this CSS file. Okay to ignore if you feel this is too big of a task.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed! This was really confusing to parse. I'll take care of this in a separate PR, so we can get this change in first. 👍

flex-shrink: 0;

// Add a left divider to the toggle button
kjellr marked this conversation as resolved.
Show resolved Hide resolved
border-radius: 0;
border-left: $border-width solid $light-gray-500;
margin-left: 1px;

&[aria-expanded="true"] .dashicon {
transform: rotate(180deg);
}
Expand Down