Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-11-22 18:25:56 +01:00
Code Issues Projects Releases Wiki Activity

Notification should not hide content according to ux

Browse Source 
Issue description:
Notification will hide some content when it popup.

According to ux requirement, we should make it not extend to the whole widh of UI, and should not hide the content.

Fix:
Use clarity alert-app-level style,it will show as ux design,and text in center too.
This commit is contained in:
Mia ZHOU 2018-07-24 17:56:46 +08:00
parent c6c5745183
commit 50678523c4
2 changed files with 4 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/ui_ng/src/app/global-message/message.component.html
View File

@ -1,4 +1,4 @@
<div [class.global-message-alert]="!isAppLevel">
<div [class.alert-app-level]="!isAppLevel">
<clr-alert [clrAlertType]="globalMessage.type" [clrAlertAppLevel]="isAppLevel" [(clrAlertClosed)]="!globalMessageOpened" (clrAlertClosedChange)="onClose()">
<div class="alert-item">
<span class="alert-text">{{message}}</span>

19
src/ui_ng/src/app/global-message/message.component.scss
View File

@ -1,16 +1,3 @@
.global-message-alert {
position: relative;
top: 0;
left: 0;
width: 100%;
z-index: 999;
margin-bottom: 20px;
}
.alert-item {
display: inline-block;
text-align: center;
}
:host >>> .alert-icon-wrapper{
display: inline;
}
:host >>> .alert.alert-app-level.alert-warning{padding: 0;}
.alert-app-level {
margin-bottom:20px;
}