Skip to content

Commit

Permalink
UX: Make the menu links header within the user panel more flexible
Browse files Browse the repository at this point in the history
  • Loading branch information
awesomerobot committed May 30, 2020
1 parent 8f59f15 commit 32e40ea
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions app/assets/stylesheets/common/base/menu-panel.scss
Original file line number Diff line number Diff line change
Expand Up @@ -279,10 +279,9 @@

div.menu-links-header {
width: 100%;
display: table;
border-collapse: separate;
border-spacing: 0 0.5em;
margin-bottom: 0.5em;
.menu-links-row {
box-sizing: border-box;
border-bottom: 1px solid dark-light-choose($primary-low, $secondary-medium);
display: flex;

Expand All @@ -295,15 +294,17 @@ div.menu-links-header {
flex-wrap: wrap;
&.user {
margin-right: auto;
flex: 1 1 0;
overflow: hidden;
}
&.glyphs {
flex-wrap: wrap;
flex-wrap: nowrap;
text-align: right;

a {
// Expand the click area a bit.
padding-left: 0.55em;
padding-right: 0.55em;
padding-left: 0.6em;
padding-right: 0.6em;
}
}

Expand Down Expand Up @@ -347,10 +348,11 @@ div.menu-links-header {
padding: 0.3em 0.5em;
}
a.user-activity-link {
box-sizing: border-box;
align-items: center;
display: flex;
margin: -0.5em 0;
max-width: 120px;
max-width: 100%;

// `overflow: hidden` on `.user-activity-link` would hide the `::after`
// pseudo element (used to create the tab-looking effect). Sets `overflow:
Expand All @@ -359,7 +361,6 @@ div.menu-links-header {

span.d-label {
display: block;
max-width: 130px;
@include ellipsis;
}

Expand Down

1 comment on commit 32e40ea

@discoursebot
Copy link

Choose a reason for hiding this comment

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

This commit has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/user-dropdown-buttons-are-wrapping/153011/6

Please sign in to comment.