-
-
Notifications
You must be signed in to change notification settings - Fork 31
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(pacer): Adds new floating banner to the Login Page #368
Conversation
This commit relocates the constants containing values used to create banner variants from the options.js file to the content.js file.
3c1ab5b
to
e3e3c21
Compare
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.
This looks good to me. Can you also make it so that clicking the button dismisses it as well, so that if people click the big button, they don't also have to find the little X?
@mlissner I tweaked the code to dismiss the banner when users click the learn more button. Here's a gif to see it in action: |
This PR addresses the latest comment from #367. It improves the user experience by relocating the banner from within the popup to a more prominent position on the PACER Login page. This change ensures the banner is clearly visible to users, promoting better awareness and interaction.
Here's a list of the key changes of this PR:
Removes the logic to display the badge and the old banner within the popup window.
The
options.html
file gets updated by removing all outdated HTML elements related to the old banner.A new helper method named
addRecapBannerToLoginPage
is introduced. This method uses Javascript to dynamically create the new banner.Enhances the content.js file with new logic to manage the banner visibility on the PACER login page.
Here's a screenshot of the login page with the banner: