Closed maryjaf closed 1 month ago
@maryjaf is attempting to deploy a commit to the Giveth Team on Vercel.
A member of the Team first needs to authorize it.
The changes in this pull request focus on the PassportBanner
component within the src/components/PassportBanner.tsx
file. Modifications include the introduction of a loading state check, which prevents rendering until necessary data is available. The rendering logic has been streamlined, and the JSX structure improved for better readability. Additionally, the PassportBannerWrapper
has been updated to a sticky
position, ensuring visibility during scrolling. Content identifiers for the formatMessage
function have been updated for clarity, while overall state management and modal handling remain unchanged.
File Path | Change Summary |
---|---|
src/components/PassportBanner.tsx | - Added loading state check to prevent rendering until data is available. - Streamlined rendering logic and improved JSX structure for readability. - Updated PassportBannerWrapper to use sticky position with top value and z-index . - Changed content identifiers for formatMessage for clarity. |
🐰 In the land of code where banners sway,
A sticky friend now leads the way.
With checks for loading, clear and bright,
It waits for data, then takes flight.
So scroll along, with joy and cheer,
ThePassportBanner
is always near! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@maryjaf on what task this PR is referring? thx
@maryjaf on what task this PR is referring? thx
https://github.com/Giveth/giveth-dapps-v2/issues/4520 @kkatusic
@maryjaf you have some prettier issues in this PR, please install in VS code editor Prettier - Code formatter and try to format code after that. After install re-open your editor.
add sticky passport banner
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Improvements