Skip to content

chore(ui5-carousel): standardize internal naming (slide→page, remove Active/Selected/Viewport)#13204

Merged
TeodorTaushanov merged 4 commits intomainfrom
carousel_naming
Mar 10, 2026
Merged

chore(ui5-carousel): standardize internal naming (slide→page, remove Active/Selected/Viewport)#13204
TeodorTaushanov merged 4 commits intomainfrom
carousel_naming

Conversation

@TeodorTaushanov
Copy link
Member

@TeodorTaushanov TeodorTaushanov commented Mar 6, 2026

Replace “slide” → “page” and correct any places where “page” should actually be “item”.
Remove Active, Selected and Viewport words.

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Mar 6, 2026

🧹 Preview deployment cleaned up: https://pr-13204--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 6, 2026 10:55 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 9, 2026 14:53 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 9, 2026 15:02 Inactive
@TeodorTaushanov TeodorTaushanov requested review from a team and Stoev March 10, 2026 07:55
Copy link

@Stoev Stoev left a comment

Choose a reason for hiding this comment

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

Please consider as alternative:

chore(ui5-carousel): standardize internal naming (slide→page, remove Active/Selected/Viewport)

@TeodorTaushanov TeodorTaushanov merged commit 85e9be1 into main Mar 10, 2026
14 checks passed
@TeodorTaushanov TeodorTaushanov deleted the carousel_naming branch March 10, 2026 14:35
@TeodorTaushanov TeodorTaushanov changed the title chore(ui5-carousel): fix internal naming chore(ui5-carousel): standardize internal naming (slide→page, remove Active/Selected/Viewport) Mar 10, 2026
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 10, 2026 14:35 Inactive
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.

4 participants