Task #95098
closedModals use semantically wrong declaration
100%
Description
I am aware of 4 wizards:
1. new content element wizard (page module)
2. translation wizard (page module)
3. new dashboard wizard
4. new form wizard
The severity of those modals and the primary buttons (if available) are declared as follows:
1. new content element wizard- modal: notice
- button: n/a
- modal: notice
- button: info
- modal: notice
- button: warning
- modal: info
- button: info
- modal: notice
- button: info
- modal: warning
- button: warning
The severities are described as follows:
- notice: non-actionable statement
- info: actionable statement
- warning: negative connotation, something will "go wrong"
- error: negative connotation, something "went already wrong"
This should be
a) fixed
b) somehow documented
Regarding a) IMHO the severity "info" would be the way to go since the modals require actions/ are actionable. As it can be seen in the new dashboard modal the severity is used for styling purposes:
This must never be the case. As we all know: form follows function (semantic) :).
That said, further adaptions to the styling of those severities could be made.
Files
Updated by Martin Kutschker about 3 years ago
Abusing semantic markup for styling is an absolute no-go when the BE ever should become accessible.
Updated by Riccardo De Contardi about 3 years ago
- Related to Task #81238: Streamline severity and message of delete modal added
Updated by Simon Schaufelberger over 2 years ago
- File form-framework.png form-framework.png added
- Description updated (diff)
Updated by Simon Schaufelberger over 2 years ago
- File dashboard.png dashboard.png added
Updated by Simon Schaufelberger over 2 years ago
- File show-columns-for-page.png show-columns-for-page.png added
- File install-tool-login.png install-tool-login.png added
- Description updated (diff)
Updated by Gerrit Code Review over 2 years ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74064
Updated by Simon Schaufelberger over 2 years ago
Updated by Anonymous over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset c3c38b3bc939c99f6733cb106adfba396a54c65a.