Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

feat(1xp-banner): add banner to /signin, /account, and /error pages #2195

Merged
merged 5 commits into from
Sep 7, 2023

Conversation

techtolentino
Copy link
Contributor

@techtolentino techtolentino commented Sep 6, 2023

Changes

Closes #2189

Implementation details

  • added q-one-xp-banner to signin.pug, userAccount.pug, and error.pug templates
  • pulled $one-xp-banner-height-* variables into the settings.scss since it's used in multiple scss files
  • updated .sign-in styles to make banner work
    • note: I had to add a 1px to the top and max-height calculations so the bottom-border of the banner would show

Screenshots

Sign in and Account pages

Screen.Recording.2023-09-06.at.10.34.12.AM.mov

Error page
Screenshot 2023-09-06 at 2 50 03 PM (2)

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

vabarbosa
vabarbosa previously approved these changes Sep 6, 2023
Copy link
Contributor

@vabarbosa vabarbosa left a 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

image

404 page

image

syllabus pages

image image

@techtolentino
Copy link
Contributor Author

@vabarbosa - I thought about these pages but didn't fully know if they should be everywhere

maybe they should?

@y4izus?

@vabarbosa
Copy link
Contributor

vabarbosa commented Sep 6, 2023

@vabarbosa - I thought about these pages but didn't fully know if they should be everywhere

at minimum i would say the 404 page should have the banner. i'm ok with leaving the other pages as is though.

@techtolentino techtolentino changed the title feat(1xp-banner): add banner to /signin and /account pages feat(1xp-banner): add banner to /signin, /account, and /error pages Sep 6, 2023
@techtolentino
Copy link
Contributor Author

at minimum i would say the 404 page should have the banner. i'm ok with leaving the other pages as is though.

@vabarbosa - good idea, updated here 1e9e6ab

Copy link
Contributor

@vabarbosa vabarbosa left a 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!

@y4izus
Copy link
Contributor

y4izus commented Sep 7, 2023

I think is enough with that pages, so I'm gonna merge

@y4izus y4izus merged commit 5d9ba88 into main Sep 7, 2023
4 checks passed
@y4izus y4izus deleted the feat/add-banner-to-signin-page branch September 7, 2023 10:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add 1XP banner to "Account" and "Sign in" pages
3 participants