Skip to content

Add or-vaadin-tabs component - #3471

Open
Ekhorn wants to merge 3 commits into
mainfrom
feature/add-tabs-component
Open

Ekhorn wants to merge 3 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 Lumo variants, sliding, rounding and a mobile app design
  • 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
@sonarqubecloud

Copy link
Copy Markdown

@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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant