PR REVIEWCSS Z-Index Stacking Context#155AtlassianNotion
00:00Mid~8 min
PULL REQUEST
Open#155 · 1 commit
CSS Z-Index Stacking Context
mid-dev-44 wants to mergefeat/smooth-page-loadmain
MI
mid-dev-44
1 file changed · 2 hours ago
YOUR MISSION
A PR added a subtle fade-in animation to the main application wrapper. Suddenly, all confirmation modals (`z-index: 9999`) are rendering *underneath* the site navigation header (`z-index: 100`).
Mid~8 min
HINTS (0/3 used)
REVIEWING AS
SR
Senior Engineer
@ Atlassian · Backend Platform
src / styles / layout.css+2 0
10
10
.app-wrapper {
11
11
display: flex;
12
12
flex-direction: column;
13
13
min-height: 100vh;
14
+
animation: fadeIn 0.3s ease-in;
+
15
+
opacity: 0.99; /* Prevents visual pop */
+
14
16
}
15
17
16
18
.modal-overlay {
17
19
position: fixed;
18
20
z-index: 9999;
19
21
}
Click any changed line (+/−) to flag an issue · or Approve if everything looks safe