Skip to content

Add or-vaadin-tabs component - #3471

Merged
MartinaeyNL merged 5 commits into
mainfrom
feature/add-tabs-component
Oct 26, 2026
Merged

MartinaeyNL merged 5 commits into
mainfrom
feature/add-tabs-component

Conversation

@Ekhorn

@Ekhorn Ekhorn commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds <or-vaadin-tabs>, a wrapper of Vaadin tabs with the Lumo look, and a sliding variant: a more mobile-friendly bar where the tabs share the width and an indicator slides behind the selected tab.

Changelog

  • Add or-vaadin-tabs, which takes <vaadin-tab> children and re-exports Tab
  • Load the Lumo tabs styles for it in @openremote/theme, so the Lumo variants apply
  • Add the theme="sliding" variant, styled through --or-tabs-* properties
  • Add stories for the sliding variant and its rounding, and link the Vaadin docs for the rest
  • Add component tests for selection, the indicator and equal-width-tabs
  • Bump the Vaadin dependencies to 25.3.3, as a newly added @vaadin/tabs resolved to 25.3.3 and pulled in a second copy of the theming package

Checklist

  • 1. Acceptance criteria of the linked issue(s) are met
  • 2. Tests are written and all tests pass
  • 3. Changes are manually tested by you and the reviewer
  • 4. Documentation is written or updated

@Ekhorn
Ekhorn requested a review from MartinaeyNL October 22, 2026 15:47
@Ekhorn Ekhorn self-assigned this Oct 22, 2026
- Drop or-vaadin-tab: or-vaadin-tabs takes vaadin-tab children
- Give or-vaadin-tabs the Lumo tabs look through the theme
- Move the sliding bar into theme="sliding" variant, with matching bar and tab rounding
- Add stories for the Lumo variants, sliding and rounding
- Add tests for the variants
@Ekhorn
Ekhorn force-pushed the feature/add-tabs-component branch from 2b4b4d1 to 57ab884 Compare October 23, 2026 07:36
@Ekhorn Ekhorn changed the title Feature/add tabs component Add or-vaadin-tabs component Oct 23, 2026
@Ekhorn
Ekhorn marked this pull request as ready for review October 23, 2026 07:54
Comment thread ui/component/or-vaadin-components/stories/or-vaadin-tabs.stories.ts Outdated
Comment thread ui/component/or-vaadin-components/stories/or-vaadin-tabs.stories.ts
@wborn wborn added the Feature Development of new feature label Oct 26, 2026
@MartinaeyNL
MartinaeyNL requested review from MartinaeyNL and removed request for MartinaeyNL October 26, 2026 12:58
Drop the stories that repeat the Vaadin style variants and link the Vaadin docs instead.
@sonarqubecloud

Copy link
Copy Markdown

@Ekhorn
Ekhorn requested a review from MartinaeyNL October 26, 2026 16:04
@MartinaeyNL
MartinaeyNL added this pull request to the merge queue Oct 26, 2026
Merged via the queue into main with commit ba1d572 Oct 26, 2026
24 checks passed
@MartinaeyNL
MartinaeyNL deleted the feature/add-tabs-component branch October 26, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Development of new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants