Project

General

Profile

Actions

Bug #102331

closed

z-index: Clear cache menu is hidden by flash messages.

Added by Christoph Lehmann 6 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2023-11-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

A flash message should not hide the clear cache menu.


Files

clearcachemenuhidden.png (83.3 KB) clearcachemenuhidden.png Christoph Lehmann, 2023-11-06 22:05

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #102459: Moved notification may break AC testsResolved2023-11-22

Actions
Related to TYPO3 Core - Bug #102691: Notifications now appear from bottom right instead of top rightClosed2023-12-18

Actions
Actions #1

Updated by Andreas Kienast 6 months ago

Personally, I think it's fine the way it is. Flash messages are temporary and disappear after a few seconds, unless it's an error. I'll redirect this to the UX team.

Actions #2

Updated by Rachel Foucard 6 months ago

Hello,

the flash message z-index is good, because it's supposed to be hover anything in the interface, but it's probably the position in the screen that is unusual, and cause this situation. Usually, a flash message or snackbar should stand at the bottom / right of the screen. With a bottom position, it is easy to see and to interact, and it is less likely to interfere with important menus or actions.

Rachel

Actions #3

Updated by Gerrit Code Review 6 months 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/+/81728

Actions #4

Updated by Marcin Sągol 6 months ago

I have proposed patch, where the alert container is rendered centered. You can test it.
In my opinion it might be better than having it aligned to the upper right corner.
There was also this topis here: https://ux.stackexchange.com/questions/39265/placement-of-flash-growl-notifications-or-messages-in-web-application

Actions #5

Updated by Gerrit Code Review 6 months ago

Patch set 2 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/+/81728

Actions #6

Updated by Gerrit Code Review 6 months ago

Patch set 3 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/+/81728

Actions #7

Updated by Rachel Foucard 6 months ago

Hello Marcin,

Thank you for working on this patch.

Centering the alert container won't solve the problem because other important action button will be hidden according the screen size. A lot of important action buttons or menu are generaly on the top of a screen.

Usually, a flash message or snackbar should stand at the bottom of the screen (bottom / right for large screen, and bottom / center for mobile screen).

Rachel

Actions #8

Updated by Gerrit Code Review 6 months ago

Patch set 4 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/+/81728

Actions #9

Updated by Gerrit Code Review 5 months ago

Patch set 5 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/+/81728

Actions #10

Updated by Gerrit Code Review 5 months ago

Patch set 6 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/+/81728

Actions #11

Updated by Marcin Sągol 5 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #12

Updated by Andreas Kienast 5 months ago

  • Related to Bug #102459: Moved notification may break AC tests added
Actions #13

Updated by Gerrit Code Review 5 months ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81897

Actions #14

Updated by Gerrit Code Review 5 months ago

Patch set 2 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81897

Actions #15

Updated by Gerrit Code Review 5 months ago

Patch set 3 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81897

Actions #16

Updated by Marcin Sągol 5 months ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Riccardo De Contardi 5 months ago

  • Related to Bug #102691: Notifications now appear from bottom right instead of top right added
Actions

Also available in: Atom PDF