blomios/IdeA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(tickets): TicketViewportSelect passe au-dessus des FloatingWindow

Browse Source 
Le popup viewport-positionné utilisait zIndex.menuDropdown (40), inférieur à
zIndex.floatingWindow (50) : il s'affichait derrière une fenêtre flottante
ouverte. Ajout de zIndex.transientPopup (65), au-dessus de floatingWindow et
floatingWindowNested, pour tout popup transitoire porté par document.body
depuis une FloatingWindow.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Blomios
2026-07-28 22:37:36 +02:00
parent 6fa41013ed
commit ec305657ad
3 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
frontend/src/features/tickets/TicketViewportSelect.test.tsx
View File

@ -40,7 +40,8 @@ describe("TicketViewportSelect", () => {
name: "filter by assignee",
});
expect(listbox.parentElement).toBe(document.body);
expect(listbox.style.zIndex).toBe(String(zIndex.menuDropdown));
expect(listbox.style.zIndex).toBe(String(zIndex.transientPopup));
expect(zIndex.transientPopup).toBeGreaterThan(zIndex.floatingWindow);
fireEvent.click(screen.getByRole("option", { name: "Alpha" }));
expect(onChange).toHaveBeenCalledWith("alpha");
@ -102,4 +103,3 @@ describe("TicketViewportSelect", () => {
expect(screen.queryByRole("listbox")).toBeNull();
});
});

2
frontend/src/features/tickets/TicketViewportSelect.tsx
View File

@ -158,7 +158,7 @@ export function TicketViewportSelect({
left: position.left,
width: position.width,
maxHeight: position.maxHeight,
zIndex: zIndex.menuDropdown,
zIndex: zIndex.transientPopup,
}}
>
{options.map((option) => (

9
frontend/src/shared/ui/zIndex.ts
View File

@ -7,9 +7,10 @@
* LayoutGrid/LeafView (G5).
*
* Consumers: `FloatingWindow` (floatingWindow), `MenuBar` dropdowns
* (menuDropdown), `TicketPicker` (floatingWindowNested), background-task toasts
* (toast). Introduced by lot B (#18) and adopted as the canonical scale by lot A
* (#16). See the sprint scoping note `ui-rework-sprint-scoping-contracts`.
* (menuDropdown), `TicketPicker` (floatingWindowNested), transient popups opened
* from floating windows (transientPopup), background-task toasts (toast).
* Introduced by lot B (#18) and adopted as the canonical scale by lot A (#16).
* See the sprint scoping note `ui-rework-sprint-scoping-contracts`.
*/
export const zIndex = {
/** Dropdown menus / popovers anchored to a control. */
@ -19,6 +20,8 @@ export const zIndex = {
/** A floating window opened from within another floating window (e.g. a
* ticket picker launched from a sprint/link dialog). Sits above it. */
floatingWindowNested: 60,
/** Viewport-positioned popups that must float above their owning window. */
transientPopup: 65,
/** Transient toasts, always on top. */
toast: 70,
} as const;