-
Notifications
You must be signed in to change notification settings - Fork 87
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
feat: add <sg-govt-banner-component> to top of public forms #1439
Conversation
Have you tested how the SG Govt masthead will look with a banner message? You can simulate a notification banner by adding
to the |
@syan-syan Yes, definitely. Just confirming that you don't want the indent to be there (move the text to the left)? Will post with screenshots once I'm done. |
Only managed to make this work by specifying the variable and value in It's easier to let the site banner stay on top because it belongs to |
May actually be fine, will just need to see if notification banners will block the masthead (when present)! Couple avenues to try:
so the masthead will always be at the top. Make sure to check if scrolling will hide the masthead (when there are info banners up) or if the padding is off. |
As discussed, will pull and update screenshots and double check behaviour |
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.
lgtm, @syan-syan for your UI approval
lgtm! |
Problem
Closes #969
Before & After Screenshots
BEFORE:
AFTER:
With info banner
Without
Deploy Notes
The indentation of the text
A Singapore Government Agency Website
looks strange on form pages.