-
Notifications
You must be signed in to change notification settings - Fork 251
feat(1xp-banner): add banner to /signin
, /account
, and /error
pages
#2195
Conversation
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.
thank you! this looks good @techtolentino
i am approving so we can merge this but
while we are here you may want to confirm if banner should be included it on the following and if so maybe open/create new issue/PR
EULA page
404 page
syllabus pages
@vabarbosa - I thought about these pages but didn't fully know if they should be everywhere maybe they should? |
at minimum i would say the 404 page should have the banner. i'm ok with leaving the other pages as is though. |
…atypus into feat/add-banner-to-signin-page
/signin
, /account
, and /error
pages
@vabarbosa - good idea, updated here 1e9e6ab |
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.
thank you @techtolentino!!!
looks good!
I think is enough with that pages, so I'm gonna merge |
Changes
Closes #2189
Implementation details
q-one-xp-banner
tosignin.pug
,userAccount.pug
, anderror.pug
templates$one-xp-banner-height-*
variables into thesettings.scss
since it's used in multiple scss files.sign-in
styles to make banner work1px
to thetop
andmax-height
calculations so the bottom-border of the banner would showScreenshots
Sign in and Account pages
Screen.Recording.2023-09-06.at.10.34.12.AM.mov
Error page
preview of sign in page: https://platypus-pr-2195.ecy0akwlcpw.us-south.codeengine.appdomain.cloud/signin
preview of error page: https://platypus-pr-2195.ecy0akwlcpw.us-south.codeengine.appdomain.cloud/test