Skip to content

feat!: Remove --experimental-css-variables option - #1484

Open
maxreichmann wants to merge 4 commits into
mainfrom
remove-experimental-css-option
Open

feat!: Remove --experimental-css-variables option#1484
maxreichmann wants to merge 4 commits into
mainfrom
remove-experimental-css-option

Conversation

@maxreichmann

@maxreichmann maxreichmann commented Jul 27, 2026

Copy link
Copy Markdown
Member

JIRA: CPOUI5FOUNDATION-1271


The SAP Theme Designer can create custom themes based on CSS variables instead of LESS variables. To establish a consistent and standardized approach to theme customization, this feature will be no longer supported in UI5 CLI. Thus with this PR, the --experimental-css-variables option will be removed for both the ui5 build and ui5 serve commands.


What has been done (for reviewing)

  • Yargs command option removed (+example) -> results in error now when used
  • ThemeBuilder API adjusted
  • "cssVariables" logic removed across repo
  • docs: breaking change info + explanation added
  • docs: existing docs adjusted
  • test: Tests covering "cssVariables" logic removed (incl. fixtures)

@maxreichmann
maxreichmann force-pushed the remove-experimental-css-option branch from 730ef0c to c899805 Compare July 28, 2026 09:27
@maxreichmann maxreichmann changed the title refactor: Remove --experimental-css-variables option feat!: Remove --experimental-css-variables option Jul 28, 2026
@maxreichmann
maxreichmann marked this pull request as ready for review July 28, 2026 10:46
@maxreichmann
maxreichmann requested a review from a team July 28, 2026 10:46

@RandomByte RandomByte left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good overall!

Comment thread packages/builder/lib/tasks/generateThemeDesignerResources.js Outdated
Comment thread packages/cli/test/lib/cli/commands/build.js Outdated
Comment thread internal/documentation/docs/updates/migrate-v5.md Outdated
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.

2 participants