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

Fix Tab selectors on smaller screens #473

Merged
merged 2 commits into from
Dec 12, 2023
Merged

Fix Tab selectors on smaller screens #473

merged 2 commits into from
Dec 12, 2023

Conversation

0xjei
Copy link
Contributor

@0xjei 0xjei commented Dec 12, 2023

Description

This PR aims to solve the problem by adding a dynamic overflowX and removing the <TabIndicator> and replicating its aesthetic behaviour with a borderBottom by styling the Tab when selected.

Related Issue

See #460

Does this introduce a breaking change?

  • Yes
  • No

Other information

Screencast.from.12-12-2023.10.57.14.webm

@0xjei 0xjei requested review from cedoor and vplasencia December 12, 2023 10:00
Copy link
Member

@cedoor cedoor left a comment

Choose a reason for hiding this comment

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

Great 👍🏽 TabIndicator must be removed to pass ESLint checks.

apps/website/src/app/learn/page.tsx Outdated Show resolved Hide resolved
Copy link
Member

@cedoor cedoor left a comment

Choose a reason for hiding this comment

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

LGTM!

@cedoor cedoor merged commit cbf66a8 into chore/website Dec 12, 2023
2 checks passed
Copy link

gitpoap-bot bot commented Dec 12, 2023

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2023 Semaphore Contributor:

GitPOAP: 2023 Semaphore Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@cedoor cedoor deleted the website/460 branch December 12, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants