Skip to content
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

make-tier-comparison-header-sticky #6151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eyeaadil
Copy link

@eyeaadil eyeaadil commented Jan 3, 2025

Description

This PR fixes #6122

Notes for Reviewers

Preview for commit:https://677805329d3a5b036e174e6e--layer5.netlify.app/

After changes
image

image

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Jan 3, 2025

🚀 Preview for commit 09c907a at: https://677805329d3a5b036e174e6e--layer5.netlify.app

@Ajay-singh1
Copy link
Member

Screenshot_20250103-210841_Chrome
Fix the borders

@eyeaadil
Copy link
Author

eyeaadil commented Jan 3, 2025

Screenshot_20250103-210841_Chrome Fix the borders

can you please specify which border you are talking about?

@Ajay-singh1
Copy link
Member

Screenshot from 2025-01-03 21-35-10

This border

Screencast.from.2024-12-31.18-53-44.mp4

And make it something like this so that when the user scrolls the header stucks to the top until it reaches the bottom of the container.

@leecalcote
Copy link
Member

Is this PR a duplicate of #6138?

@Venkata-Sai-Vishwanath-robo

This PR looks pretty neat, but there is an extra space at the end of the table. This space grows larger as the window size decreases, which might misleadingly suggest that something hasn't rendered correctly or is still loading. A better use of this space could be to place call-to-action buttons there as well. Positioning the buttons in this spot would make it more convenient for users to access the free trial option without needing to scroll back to the top.
The call-to-action buttons could be added after the approval from @leecalcote

image

image

@leecalcote leecalcote requested a review from SAHU-01 January 5, 2025 15:48
@Venkata-Sai-Vishwanath-robo

Screenshot from 2025-01-03 21-35-10

This border

Screencast.from.2024-12-31.18-53-44.mp4
And make it something like this so that when the user scrolls the header stucks to the top until it reaches the bottom of the container.

@Ajay-singh1 I like the idea of having the header stick to the top, as shown in the video. However, don’t you think the table's content, including the expanded sections, might be too extensive? This could force users to scroll through everything, even if they’re not interested.

@Ajay-singh1
Copy link
Member

@Venkata-Sai-Vishwanath-robo Yeah! I agree with you.

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

Successfully merging this pull request may close these issues.

[Pricing] Make Tier Comparison Header Sticky
5 participants