-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
Global announcement #71
Conversation
There is no funky "User has read an accepted it and thus hides the message"-behavior implemented. This would be more complex and raise the question about lifetime of the announcement. |
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.
looks good to me. Only thing to mention is that fixed sizes in px should be prevented if possible or set to a global constant.
box-shadow: | ||
0 1px 1px 0 rgba(0, 0, 0, 0.2), | ||
0 2px 10px 0 rgba(0, 0, 0, 0.16); | ||
border-radius: 4px; |
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 there a constant available for the border-radius?
|
||
#login-global-announcement { | ||
width: 100%; | ||
max-width: 520px; |
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.
Prevent fixed px.
position: relative; | ||
z-index: 1; | ||
} | ||
|
||
#login-card { | ||
width: 100%; | ||
max-width: 520px; |
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.
avoid sizing in fixed px
Adds a configurable, highlighted message to the Login- and Home-Screen.
Closes #63
A downport to RC core is pending. I'll create a new issue for that on RC