[ui5-dialog]: compact density has no effect on content padding

[rdord]

Bug Description

ui5-dialog (and all popups using PopupsCommon-parameters.css ) ignores compact density when applying content padding. The media-range-based padding vars (--_ui5_popup_content_padding_m_l: 2rem, --_ui5_popup_content_padding_xl: 3rem) are not overridden in compact mode, so a wide dialog under .ui5-content-density-compact still renders 32px horizontal padding instead of 16px.

23 other components in the repo have a @container style(--ui5_content_density: compact) block in their *-parameters.css file. PopupsCommon-parameters.css is missing it.

Affected Component

ui5-dialog / ui5-popup

Expected Behaviour

Under .ui5-content-density-compact, dialog content padding should be 1rem (16px) at all breakpoints, consistent with Fiori compact spec and with how other components handle density.

Isolated Example

https://stackblitz.com/edit/js-t13sqby5?file=index.html

Steps to Reproduce

1. Add .ui5-content-density-compact to a parent element
2. Open a ui5-dialog that is wide enough to get media-range="M" or "L"
3. Inspect computed padding on .ui5-popup-content — it renders 32px instead of 16px

Log Output, Stack Trace or Screenshots

No response

Priority

Low

UI5 Web Components Version

2.26.0

Browser

Chrome

Operating System

No response

Additional Context

ViewSettingsDialog-parameters.css in the same repo correctly implements the compact override pattern.

Proposed fix, add to packages/main/src/themes/base/PopupsCommon-parameters.css:

@container style(--ui5_content_density: compact) {
  :host {
    --_ui5_popup_content_padding_m_l: 1rem;
    --_ui5_popup_content_padding_xl: 1rem;
    --_ui5_popup_header_footer_padding_m_l: 1rem;
    --_ui5_popup_header_footer_padding_xl: 1rem;
  }
}

Organization

No response

Declaration

• I’m not disclosing any internal or sensitive information.

[unazko]

Hello @UI5/ui5-team-ctr-rod, forwarding this bug to you.

Analysis summary:
ui5-dialog (and all popups sharing PopupsCommon-parameters.css) ignores .ui5-content-density-compact for the media-range-based padding variables (--_ui5_popup_content_padding_m_l, --_ui5_popup_content_padding_xl). The compact override block using @container style(--ui5_content_density: compact) is missing from that file, while 23 other components in the repo implement it correctly.

Verified: Tested with the author's StackBlitz sample (https://stackblitz.com/edit/js-t13sqby5?file=index.html) following the provided steps — the behavior is reproducible as described on screen sizes M and larger, where 32px padding is rendered instead of the expected 16px under compact density.

The reporter has also provided a ready-made CSS fix.

Affected file: packages/main/src/themes/base/PopupsCommon-parameters.css

Best regards,
Boyan