Skip to content

fix(UnderlinePanels): add explicit type="button" to UnderlinePanels - #4970

Merged
joshblack merged 5 commits into
mainfrom
chore/enable-react-button-has-type-rule
Sep 18, 2024
Merged

fix(UnderlinePanels): add explicit type="button" to UnderlinePanels#4970
joshblack merged 5 commits into
mainfrom
chore/enable-react-button-has-type-rule

Conversation

@joshblack

Copy link
Copy Markdown
Member

Closes ##4912

Changelog

New

Changed

  • Enable react/button-has-type rule
  • Refactor tests that include button elements without a type
  • Update UnderlinePanels so that the tab has an explicit type

Removed

Rollout strategy

  • Patch release

Testing & Reviewing

  • Verify that the button element with role="tab" has type="button"

@changeset-bot

changeset-botBot commented Sep 16, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: cceb471

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
NameType
@primer/reactPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@joshblack
joshblack requested a review from a team as a code ownerSeptember 16, 2024 17:00
@github-actionsgithub-actionsBot added the staff Author is a staff member label Sep 16, 2024
@github-actions
github-actionsBottemporarily deployed to storybook-preview-4970 September 16, 2024 17:03 Inactive
@github-actions

github-actionsBot commented Sep 16, 2024

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize
packages/react/dist/browser.esm.js97.5 KB (0%)
packages/react/dist/browser.umd.js97.67 KB (0%)

@github-actions
github-actionsBottemporarily deployed to storybook-preview-4970 September 18, 2024 16:20 Inactive
@joshblack
joshblack added this pull request to the merge queueSep 18, 2024
Merged via the queue into main with commit 44a3dc9Sep 18, 2024
@joshblack
joshblack deleted the chore/enable-react-button-has-type-rule branch September 18, 2024 16:48
@primerprimerBot mentioned this pull request Sep 18, 2024
TylerJDev pushed a commit that referenced this pull request Sep 23, 2024
…4970)
* refactor(eslint): enable react/button-has-type rule
* fix(UnderlinePanels): add explicit type="button" to UnderlinePanels
* chore: add changeset
* chore: fix eslint violation
---------
Co-authored-by: Josh Black <joshblack@users.noreply.github.com>
@primerprimerBot mentioned this pull request Oct 18, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

staffAuthor is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@joshblack@broccolinisoup