Conversation
|
Preview URLs (comment last updated: 2023-01-10 16:22:00) |
dipikabh
left a comment
dipikabh
left a comment
There was a problem hiding this comment.
Hello @zuzexx, the author in the issue is referring to the list in the Description section, under the sentence "To better understand the situation, here is the stacking context hierarchy:". This list contains the three dots for illustration purposes.
Even if you try to improve the code snippets, I believe the list with three dots, which is causing the confusion, still exists.
Could you make corresponding updates to the list to remove the three dots and also to match your new example?
|
I can do the changes. Just that I understand correctly, you want me to change the three dots between level 2 divs to the appropriate amount of divs so that it reflects the new example? |
dipikabh
left a comment
dipikabh
left a comment
There was a problem hiding this comment.
Perhaps, we don't need to remove the ellipsis. We can clarify the use of ellipsis in the description.
How about updating the following line as:
line 28: "To better understand the situation, the following example illustrates the stacking context hierarchy (the three dots "..." represent multiple repetition of the previous line):
|
|
||
| - LEVEL #2 (`z-index`: 1) | ||
| - … | ||
| - LEVEL #2 (`z-index`: 1) |
There was a problem hiding this comment.
| - LEVEL #2 (`z-index`: 1) |
let's keep the ellipsis
|
|
||
| - LEVEL #1 | ||
| - … | ||
| - LEVEL #1 |
There was a problem hiding this comment.
| - LEVEL #1 |
let's keep the ellipsis
There was a problem hiding this comment.
Ok, thank you for clarification. I will get to it once I come home.
2 participants
Description
I added a few more level 2 divs, in order to better illustrate the example.
Motivation
The example is more understandable, based on the description above the code.
Related issues and pull requests
Fix #23274