Skip to content

Conversation

@darshitdudhaiya
Copy link

Description

This PR fixes the sticky behavior of the currency dropdown in the Pricing page. The dropdown now scrolls smoothly with the page and no longer remains fixed when the user scrolls.

Notes for Reviewers

  • Verified that the dropdown menu moves correctly on scroll for all screen sizes.
  • Cleaned up MenuProps in the CurrencySelect component for better readability and maintainability.
  • Ensured that the dropdown styling and positioning remain consistent with the design.
  • No breaking changes to other components.

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Oct 9, 2025

🚀 Preview for commit 38c616b at: https://68e83ca724b3759775a25244--layer5.netlify.app

@darshitdudhaiya
Copy link
Author

@ChetanFTW can you review this PR and let me know if there is any changes needed.

@l5io
Copy link
Contributor

l5io commented Oct 10, 2025

🚀 Preview for commit 51950b9 at: https://68e8f8b59944789f89c5a596--layer5.netlify.app

@ChetanFTW
Copy link
Member

you can check there is still sticky dropdown
Screenshot from 2025-10-10 13-54-54

@darshitdudhaiya
Copy link
Author

you can check there is still sticky dropdown Screenshot from 2025-10-10 13-54-54

sure i'll solve it.

@leecalcote leecalcote added the issue/dco Commit sign-off instructions label Oct 10, 2025
@github-actions
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

1 similar comment
@github-actions
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

@l5io
Copy link
Contributor

l5io commented Oct 10, 2025

🚀 Preview for commit 475bfac at: https://68e95f3e91eef310a44a31c0--layer5.netlify.app

@darshitdudhaiya darshitdudhaiya force-pushed the bug-fix/currency-selector-dropdown branch from 475bfac to 8af0a4f Compare October 10, 2025 20:49
@l5io
Copy link
Contributor

l5io commented Oct 10, 2025

🚀 Preview for commit 8af0a4f at: https://68e97472889a46abcad11ecc--layer5.netlify.app

Copy link
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DCO Failing ⚠️ @darshitdudhaiya
To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~5 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin bug-fix/currency-selector-dropdown

Lakshay-Pareek and others added 18 commits October 11, 2025 21:34
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Lakshay Pareek <hopesalive0291@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: darshitdudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
… IntersectionObserver

Signed-off-by: darshitdudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Divyanshi Pathak <divyanshipathakqc@mpgi.edu.in>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Introduced AGENTS.md as a universal standard for guiding AI coding agents, consolidating project context and configuration into a single file.

Signed-off-by: Lee Calcote <lee.calcote@layer5.io>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
RashRAJ and others added 3 commits October 11, 2025 21:34
Signed-off-by: Rasheedat Atinuke Jamiu <Hartinuke16@gmail.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Rasheedat Atinuke Jamiu <56790407+RashRAJ@users.noreply.github.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
Signed-off-by: Rasheedat Atinuke Jamiu <56790407+RashRAJ@users.noreply.github.com>
Signed-off-by: Darshit Dudhaiya <darshitdudhaiya201@gmail.com>
@darshitdudhaiya darshitdudhaiya force-pushed the bug-fix/currency-selector-dropdown branch from 8af0a4f to 8c2caf0 Compare October 11, 2025 16:04
Signed-off-by: Darshit <92248308+darshitdudhaiya@users.noreply.github.com>
@l5io
Copy link
Contributor

l5io commented Oct 11, 2025

🚀 Preview for commit bf8e82c at: https://68ea83b00b2eebc1d36886bb--layer5.netlify.app

@darshitdudhaiya
Copy link
Author

@ChetanFTW and @Rajesh-Nagarajan-11, can you please review this? I believe I have fully resolved the issue now.

@Namanv0509
Copy link
Member

@darshitdudhaiya
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

@simihablo simihablo requested a review from Namanv0509 October 15, 2025 23:09
@darshitdudhaiya
Copy link
Author

@darshitdudhaiya Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT

Add it as an agenda item to the meeting minutes, if you would :)

Hey @Namanv0509 , I added it to the agenda after you mentioned it, but I see this PR hasn't been merged yet. Are there any problems or issues? I'm ready to complete it again if needed.

@Rajesh-Nagarajan-11
Copy link
Member

Rajesh-Nagarajan-11 commented Oct 18, 2025

@darshitdudhaiya did you discuss this in meeting ? And get feedback on this PR

@Rajesh-Nagarajan-11
Copy link
Member

@ChetanFTW FYI

@ChetanFTW
Copy link
Member

On Currency Dropdown is now responsive and LGTM
hey @Namanv0509 PTAL, I this PR the dropdown will automatically closed while overlapes widget and In my PR it will block the widgets until user make the section

@darshitdudhaiya
Copy link
Author

@darshitdudhaiya did you discuss this in meeting ? And get feedback on this PR

No, I just added this as an agenda item in the meeting minutes.

@Rajesh-Nagarajan-11
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT
Add it as an agenda item to the meeting minutes, if you would 😄

@darshitdudhaiya
Copy link
Author

Thank you for your contribution! Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT Add it as an agenda item to the meeting minutes, if you would 😄

@Rajesh-Nagarajan-11 I've added it to the agenda and will try to join the website call today if possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/blog New posts or new blog functionality area/handbook issue/dco Commit sign-off instructions project/kanvas project/sistent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants