Skip to content

Commit

Permalink
[57092] Right side of the instance upper banner looks misaligned (#16737
Browse files Browse the repository at this point in the history
)

* replace bell icon with bell octicon

* set auto width for the header button

* remove line-height for menu item action

* remove additional space

* fix jumping icons

* undo last commit
  • Loading branch information
bsatarnejad authored Sep 18, 2024
1 parent 4818a30 commit e8e6367
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
data-test-selector="op-ian-bell"
[href]="notificationsPath()"
>
<op-icon icon-classes="icon-bell"></op-icon>
<svg bell-fill-icon size="small"></svg>
<ng-container *ngIf="(unreadCount$ | async) as unreadCount">
<span
*ngIf="unreadCount !== 0"
Expand Down
3 changes: 3 additions & 0 deletions frontend/src/app/shared/components/icon/icon.module.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import {
StarFillIconComponent,
StarIconComponent,
XIconComponent,
BellFillIconComponent,
OpGitlabIssueClosedIconComponent,
OpGitlabLabelsIconComponent,
OpGitlabIssueOpenIconComponent,
Expand Down Expand Up @@ -81,6 +82,7 @@ import {
StarFillIconComponent,
StarIconComponent,
XIconComponent,
BellFillIconComponent,

OpGitlabIssueClosedIconComponent,
OpGitlabLabelsIconComponent,
Expand Down Expand Up @@ -136,6 +138,7 @@ import {
StarFillIconComponent,
StarIconComponent,
XIconComponent,
BellFillIconComponent,

OpGitlabIssueOpenIconComponent,
OpGitlabIssueClosedIconComponent,
Expand Down
1 change: 0 additions & 1 deletion frontend/src/global_styles/common/header/app-menu.sass
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@
justify-content: center
align-items: center
height: var(--header-height)
line-height: var(--header-height)
zoom: 1
color: var(--header-item-font-color)
font-size: var(--header-item-font-size)
Expand Down
1 change: 1 addition & 0 deletions frontend/src/global_styles/primer/_overrides.sass
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ sub-header,
.Button.op-app-header--primer-button,
.Button.op-app-header--primer-button .Button-visual
color: var(--header-item-font-color)
width: auto

&:hover
background: var(--header-item-bg-hover-color)
Expand Down

0 comments on commit e8e6367

Please sign in to comment.