-
Notifications
You must be signed in to change notification settings - Fork 36
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(staff page)-switch layout with ccip #425
Conversation
add a custom-layout and modify page staff
✅ Deploy Preview for ephemeral-sable-89c7e0 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Gotta figure out how to switch the layout with token, and which CCIP token meant to be the key. |
add if/else to switch layout with ccip in query
related feature that related to #299
font-size: 20px; | ||
word-spacing: 1px; | ||
-ms-text-size-adjust: 100%; | ||
-webkit-text-size-adjust: 100%; | ||
-moz-osx-font-smoothing: grayscale; | ||
-webkit-font-smoothing: antialiased; | ||
box-sizing: border-box; |
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.
Minor: 如果可以用 tailwind 的語法代換,可以直接用。太複雜就維持現這樣
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.
好建議! 不過我的tailwind還不夠純熟. 先維持這樣. 之後再開一個PR改語法
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
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
Please modify the PR based on David's comments.
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
Description
Add a custom layout "ccip" to hide footer and header at page staff, switch the layout based on CCIP value
Steps to Test This Pull Request
Expected behavior
When ccip is true, footer and header are hidden as picture:
If false:
Related Issue
#413