-
Notifications
You must be signed in to change notification settings - Fork 63
fix(account-sidebar): fix account long name are above the other label… #1506
Conversation
…s in account sidebar
Pull Request Test Coverage Report for Build 1517
💛 - Coveralls |
@@ -12,26 +10,22 @@ | |||
> | |||
<mat-expansion-panel-header> | |||
<mat-panel-title> | |||
{{ user.firstname }} {{ user.lastname }} | |||
<div class="mat-tab-tooltip" [matTooltip]="user.firstname + user.lastname"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</mat-panel-title> | ||
<mat-panel-description> | ||
</mat-panel-description> | ||
<mat-panel-description> </mat-panel-description> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove mat-panel-description if it's not useful.
@@ -12,26 +10,22 @@ | |||
> | |||
<mat-expansion-panel-header> | |||
<mat-panel-title> | |||
{{ user.firstname }} {{ user.lastname }} | |||
<div class="mat-tab-tooltip" [matTooltip]="user.firstname + user.lastname"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need the mat-tab-tooltip class in the first place?
@@ -0,0 +1,11 @@ | |||
.mat-expansion-panel-header { | |||
max-width: 100%; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like it doesn't do anything
overflow: hidden; | ||
text-overflow: ellipsis; | ||
white-space: nowrap; | ||
max-width: 95%; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is max-width: 95%; necessary?
max-width: 100%; | ||
} | ||
|
||
.entity-card-title { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can replace entity-card-title with truncate from app.scss
…s in account sidebar