Skip to content

Commit 3fd9958

Browse files
authored
Merge pull request #2399 from visualize-admin/feat/always-show-scroll-buttons-when-needed
fix: Always show chart tab scroll buttons when needed
2 parents e7f2fd9 + 8de1afd commit 3fd9958

2 files changed

Lines changed: 7 additions & 8 deletions

File tree

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,9 @@ You can also check the
2727
filters section
2828
- Dataset browse view is now correctly displayed on mobile devices
2929
- Database-related actions are now hidden in preview mode (copy URL, share)
30-
- Chart titles in user profile are now correctly rendered
3130
- Styles
31+
- Scroll buttons in the chart selection tabs are now always shown when needed
32+
- Chart titles in user profile are now correctly rendered
3233
- Dataset tags can now wrap
3334
- Security
3435
- Added additional protection against data source URL injection

app/components/chart-selection-tabs.tsx

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -456,13 +456,11 @@ const TabsInner = ({
456456
style={{ cursor: disabled ? "auto" : "pointer" }}
457457
variant="text"
458458
>
459-
{disabled ? null : (
460-
<Icon
461-
name={
462-
direction === "left" ? "chevronLeft" : "chevronRight"
463-
}
464-
/>
465-
)}
459+
<Icon
460+
name={
461+
direction === "left" ? "chevronLeft" : "chevronRight"
462+
}
463+
/>
466464
</Button>
467465
)}
468466
>

0 commit comments

Comments
 (0)