spectrum-css icon indicating copy to clipboard operation
spectrum-css copied to clipboard

[Dialog] spectrum-Dialog--dismissable and spectrum-Dialog--fullscreen* are not compatible

Open Westbrook opened this issue 5 years ago • 1 comments

Description

Usage of .spectrum-Dialog--dismissable and its button.spectrum-Dialog-closeButton element are not compatible with .spectrum-Dialog--fullscreen* and it should be documented as such.

Currently, there are no documented usage of this combination. However, when users use these in concert the patterns are delivered in a less than palatable fashion. https://github.com/adobe/spectrum-web-components/issues/1088 We should explicitly outline that these should not be used together.

Link to documentation

https://opensource.adobe.com/spectrum-css/dialog.html

Additional context

We might also want to discuss defensively coding them out of the system. This is done in the latest code for the way that .spectrum-Dialog--dismissable will not allow the presentation of .spectrum-Dialog-buttonGroup content.

Westbrook avatar Jan 14 '21 18:01 Westbrook

Uhg, even the fullscreen pattern is considered a contribution, it's a relic from years past. Dialog needs an overhaul...

lazd avatar Jan 14 '21 23:01 lazd

Updated link: https://opensource.adobe.com/spectrum-css/?path=/docs/components-dialog--docs

The ask here at this time is to update the documentation to explicitly state that dismissible is not allowed to be used in conjunction with the fullscreen versions of the Dialog.

castastrophe avatar Dec 10 '24 20:12 castastrophe

Fixed via https://github.com/adobe/spectrum-css/pull/3347

najikahalsema avatar Feb 11 '25 18:02 najikahalsema