Skip to content

Commit

Permalink
V 3/tabs feedback (#1315)
Browse files Browse the repository at this point in the history
  • Loading branch information
svenvoskamp authored Sep 7, 2023
1 parent 52d6160 commit 920c8b3
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 21 deletions.
36 changes: 19 additions & 17 deletions packages/ui/src/composites/wui-tabs/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,27 +73,29 @@ export class WuiTabs extends LitElement {
const activeBtnBounds = activeBtn?.getBoundingClientRect()
const activeBtnTextBounds = activeBtnText?.getBoundingClientRect()

if (passiveBtn && passiveBtnText && !initialAnimation) {
if (passiveBtn && passiveBtnText && !initialAnimation && index !== this.activeTab) {
animate(passiveBtnText, { opacity: 0 }, { duration: 0.25 })
animate(passiveBtn, { width: `20px` }, { duration: 0.25, delay: 0.05 })
animate(passiveBtn, { width: '34px' }, { duration: 0.5 })
}

if (activeBtn && activeBtnBounds && activeBtnTextBounds && activeBtnText) {
this.localTabWidth = `${Math.round(
activeBtnBounds.width + activeBtnTextBounds.width + 6 + 12
)}px`

animate(
activeBtn,
{ width: `${activeBtnBounds.width + activeBtnTextBounds.width + 6}px` },
{ duration: initialAnimation ? 0 : 0.5, delay: initialAnimation ? 0 : 0.1 }
)

animate(
activeBtnText,
{ opacity: 1 },
{ duration: initialAnimation ? 0 : 0.25, delay: initialAnimation ? 0 : 0.15 }
)
if (index !== this.activeTab || initialAnimation) {
this.localTabWidth = `${
Math.round(activeBtnBounds.width + activeBtnTextBounds.width) + 6
}px`

animate(
activeBtn,
{ width: `${activeBtnBounds.width + activeBtnTextBounds.width}px` },
{ duration: initialAnimation ? 0 : 0.5 }
)

animate(
activeBtnText,
{ opacity: 1 },
{ duration: initialAnimation ? 0 : 0.25, delay: initialAnimation ? 0 : 0.05 }
)
}
}
}
}
Expand Down
9 changes: 5 additions & 4 deletions packages/ui/src/composites/wui-tabs/styles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,14 @@ export default css`
}
:host([data-type='flex'])::before {
transform: translateX(calc(var(--local-tab) * var(--local-tab-width) / 1.85));
left: 3px;
transform: translateX(calc((var(--local-tab) * 34px) + (var(--local-tab) * 4px)));
}
:host([data-type='flex']) {
display: flex;
padding: 0px 12px;
gap: 28px;
padding: 0px 0px 0px 12px;
gap: 4px;
}
:host([data-type='flex']) > button > wui-text {
Expand Down Expand Up @@ -64,7 +65,7 @@ export default css`
}
:host([data-type='flex']) > button {
width: 20px;
width: 34px;
position: relative;
display: flex;
justify-content: flex-start;
Expand Down

3 comments on commit 920c8b3

@vercel
Copy link

@vercel vercel bot commented on 920c8b3 Sep 7, 2023

Choose a reason for hiding this comment

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

@vercel
Copy link

@vercel vercel bot commented on 920c8b3 Sep 7, 2023

Choose a reason for hiding this comment

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

@vercel
Copy link

@vercel vercel bot commented on 920c8b3 Sep 7, 2023

Choose a reason for hiding this comment

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

Please sign in to comment.