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(NextPrev): offset and hompage conditions #765

Merged
merged 3 commits into from
Mar 5, 2020

Conversation

sstrubberg
Copy link
Member

@sstrubberg sstrubberg commented Mar 5, 2020

Closes #746

Now with 90% less commit garbage ;)

Added logic and style for the first button to be hidden if it's the homepage, thus giving the offset.

Blocker: It thinks every page is the homepage, displaying the next, but hiding the previous.

@sstrubberg sstrubberg requested review from a team, vpicone and jnm2377 and removed request for a team March 5, 2020 17:11
@vercel
Copy link

vercel bot commented Mar 5, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/carbon-design-system/gatsby-theme-carbon/2c647mpad
✅ Preview: https://gatsby-theme-carbon-git-fork-sstrubberg-fix-nextprev.carbon-design-system.now.sh

@sstrubberg sstrubberg changed the title added logic and style fix(NextPrev): offset and hompage conditions WIP Mar 5, 2020
@sstrubberg sstrubberg changed the title fix(NextPrev): offset and hompage conditions WIP fix(NextPrev): offset and hompage conditions Mar 5, 2020
@sstrubberg sstrubberg requested a review from vpicone March 5, 2020 19:01
Copy link
Contributor

@vpicone vpicone left a comment

Choose a reason for hiding this comment

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

Boom

@vpicone vpicone merged commit 2f8dbf3 into carbon-design-system:master Mar 5, 2020
@sstrubberg sstrubberg deleted the fix-nextprev branch March 5, 2020 23:20
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.

fix(homepage):NextPrev
2 participants