Skip to content

Conversation

@roggenkemper
Copy link
Member

@roggenkemper roggenkemper commented Jul 25, 2024

this pr adds a new variant of tabs - "floating" , which will be used on the updated issue details page

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 25, 2024
@codecov
Copy link

codecov bot commented Jul 25, 2024

Codecov Report

Attention: Patch coverage is 30.00000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 78.12%. Comparing base (659154c) to head (eff6a7e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #75029      +/-   ##
==========================================
- Coverage   78.12%   78.12%   -0.01%     
==========================================
  Files        6745     6746       +1     
  Lines      301168   301177       +9     
  Branches    51823    51825       +2     
==========================================
+ Hits       235289   235290       +1     
- Misses      59555    59563       +8     
  Partials     6324     6324              
Files Coverage Δ
static/app/components/tabs/tabList.tsx 88.88% <ø> (ø)
static/app/components/tabs/tab.tsx 71.92% <30.00%> (-9.71%) ⬇️

... and 4 files with indirect coverage changes

@roggenkemper roggenkemper marked this pull request as ready for review July 26, 2024 16:59
@roggenkemper roggenkemper requested a review from a team July 26, 2024 23:02
@roggenkemper roggenkemper enabled auto-merge (squash) July 29, 2024 18:20
@roggenkemper roggenkemper merged commit 636b10d into master Jul 29, 2024
@roggenkemper roggenkemper deleted the roggenkemper/newtabvariant branch July 29, 2024 18:23
<p>You can also use the "floating" variant, which is used below</p>
<SizingWindow>
<Tabs>
<TabList variant="floating" hideBorder>
Copy link
Member

@ryan953 ryan953 Jul 29, 2024

Choose a reason for hiding this comment

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

This could use a Matrix to better show the different values along side each other, i'd use Matrix to compate with hideBorder and also with orientation

roggenkemper added a commit that referenced this pull request Jul 29, 2024
this pr uses the new tab design introduced in
#75029 for the streamlined event
navigation component
@github-actions github-actions bot locked and limited conversation to collaborators Aug 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants