refactor: tabs + usage#1105
Merged
jorgemoya merged 1 commit intobigcommerce:refactor/ui-componentsfrom Jul 19, 2024
Merged
Conversation
|
@drewpledger is attempting to deploy a commit to the BigCommerce Platform Team on Vercel. A member of the Team first needs to authorize it. |
|
| <NextIntlClientProvider locale={locale} messages={{ Account: messages.Account ?? {} }}> | ||
| <AccountStatusProvider> | ||
| <h1 className="my-6 my-8 text-4xl font-black lg:my-8 lg:text-5xl">{t('heading')}</h1> | ||
| <h1 className="my-8 text-4xl font-black lg:my-8 lg:text-5xl">{t('heading')}</h1> |
Contributor
There was a problem hiding this comment.
do you mind to remove heading for tab layout if we do refactoring? Turner's updated figma and we don't need it anymore on Tabs.
Contributor
There was a problem hiding this comment.
We're merging to a branch, will rebase when we have all changes ready to be merged in a final PR to main.
drewpledger
force-pushed
the
dp/tabs-refactor
branch
3 times, most recently
from
3d5dd34 to
448f88f
Compare
jorgemoya
approved these changes
Jul 19, 2024
drewpledger
force-pushed
the
dp/tabs-refactor
branch
2 times, most recently
from
6e0d51e to
7c8563e
Compare
drewpledger
force-pushed
the
dp/tabs-refactor
branch
from
7c8563e to
3d4c204
Compare
deini
pushed a commit
that referenced
this pull request
Jul 19, 2024
jorgemoya
pushed a commit
that referenced
this pull request
Jul 30, 2024
bookernath
pushed a commit
that referenced
this pull request
Jul 30, 2024
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Jul 30, 2024
* refactor: tag + usages (#1107) * refactor(core): radio-group (#1106) * fix: merge with main (#1105) * refactor: remove file-chooser, no usages (#1110) * refactor: carousel (#1109) * refactor: badge (#1108) * fix: eslint issues in tabs and badge * refactor: gallery refactor (#1104) * fix(core): lint issues from gallery refactor * refactor(core): slideshow (#1111) * refactor: message component (#1112) * refactor: badge takes children (#1114) * chore(core): delete skeleton (#1113) * refactor: modal (#1115) * refactor: button (#1116) * fix: render cart count in badge * refactor: datepicker uses popover primitives (#1118) * refactor: accordions component (#1101) * refactor(core): product card (#1117) * refactor: input (#1131) * refactor: textarea (#1134) * refactor: checkbox (#1135) * refactor: date picker (#1130) * fix: radio group error state in pdp (#1133) * refactor: pick list (#1139) * fix: forward ref to button (#1140) * refactor: select (#1143) * fix: use correct font weight in badge (#1151) * refactor: swatch (#1144) * refactor: rectangle list (#1145) * refactor: pick list props (#1146) * refactor: radio group props (#1147) * refactor: breadcrumbs (#1148) * refactor: footer (#1154) * fix: forward DatePicker ref (#1155) * refactor: label (#1153) * refactor: blog post card (#1152) * refactor: rating (#1150) * refactor: sheet (#1149) * refactor: counter (#1136) * refactor: header & navigation menu (#1160) * fix: accordions props (#1162) * fix: remove transparent overlay prop in Sheet (#1163) * fix: gallery props (#1164) * feat: add pageSize prop to carousels (#1165) * chore: add changeset * chore: update changeset * fix: missing padding for sm viewport in quick search * fix: close mobile nav on navigation * fix: add aria-label to account tabs * fix: skip blog post card regression * fix: p wrapper for cart link * fix: update header visual regression test * fix: correct capitlization for tabs * fix: change hover for on click in test * fix: swatch test * fix: changeset examples * feat: add label prop to tabs * fix: update account and register spec * fix: update tests with latest changes --------- Co-authored-by: Matthew Volk <matt.volk@bigcommerce.com> Co-authored-by: Drew Pledger <drew@makeswift.com> Co-authored-by: Daniel Almaguer <daniel.almaguer@bigcommerce.com> Co-authored-by: Matthew Volk <volkmattj@gmail.com> Co-authored-by: Anudeep Vattipalli <anudeep.vattipalli@bigcommerce.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.