-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Pricing/currency dropdown #7024
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: ChetanFTW <[email protected]>
Signed-off-by: ChetanFTW <[email protected]>
|
🚀 Preview for commit fb32c0b at: https://68ec10d291eef320034a30f4--layer5.netlify.app |
|
@Namanv0509 is this change needed any longer? |
|
Yes the sticky behaviour is still there, so we need that to be fixed , but this PR wont work , as there are many changes no required , some of which can take us back also this should be handled naturally by the sistent component as it is doing in cloud.layer5.io // @ChetanFTW . |
|
@ChetanFTW any update on the following ? |
|
I did some changes using sis but doesn't works out, |
|
@Namanv0509 our sistent doesn't supports that , it's not sticky by default |
|
🚀 Preview for commit 0f6eaad at: https://68f428c2e34746daecd252d8--layer5.netlify.app |
|
🚀 Preview for commit 2330ade at: https://68f5bd5d1a0b7c721542b20d--layer5.netlify.app |
|
Is this issue resolved or can I work on it ? |
|
yes, @PARTH-TUSSLE this issue resolved |
|
🚀 Preview for commit bd4905d at: https://690cf160ebf4ddefe6cd31ee--layer5.netlify.app |
|
🚀 Preview for commit 05a91b8 at: https://6910ddd98529e78a34cec30c--layer5.netlify.app |
|
🚀 Preview for commit 55d0a34 at: https://691a1fbd9bc0597876ae8f2d--layer5.netlify.app |
Description
This is issue #6951
On this Page there is the dropdown while click to currency
Notes for Reviewers
Signed commits