Skip to content

Commit 82c463f

Browse files
fix: Remove redundant zindex values (#1257)
* sane zindex values * Removed z-indices * Reverted ShadCN example --------- Co-authored-by: matthewlipski <[email protected]> Co-authored-by: Matthew Lipski <[email protected]>
1 parent 6c89ca5 commit 82c463f

File tree

6 files changed

+55
-65
lines changed

6 files changed

+55
-65
lines changed

examples/03-ui-components/03-formatting-toolbar-block-type-items/Alert.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ export const Alert = createReactBlockSpec(
7373
return (
7474
<div className={"alert"} data-alert-type={props.block.props.type}>
7575
{/*Icon which opens a menu to choose the Alert type*/}
76-
<Menu withinPortal={false} zIndex={999999}>
76+
<Menu withinPortal={false}>
7777
<Menu.Target>
7878
<div className={"alert-icon-wrapper"} contentEditable={false}>
7979
<Icon
Lines changed: 26 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,40 @@
11
.slash-menu {
2-
z-index: 9999;
2+
background-color: white;
3+
border: 1px solid lightgray;
4+
border-radius: 2px;
5+
box-shadow: 0 0 8px #dddddd;
36

4-
background-color: white;
5-
border: 1px solid lightgray;
6-
border-radius: 2px;
7-
box-shadow: 0 0 8px #dddddd;
7+
display: flex;
8+
flex-direction: column;
9+
gap: 8px;
10+
height: fit-content;
11+
max-height: 100%;
812

9-
display: flex;
10-
flex-direction: column;
11-
gap: 8px;
12-
height: fit-content;
13-
max-height: 100%;
13+
overflow: auto;
1414

15-
overflow: auto;
15+
padding: 8px;
1616

17-
padding: 8px;
18-
19-
top: 8px;
17+
top: 8px;
2018
}
2119

2220
.slash-menu-item {
23-
background-color: white;
24-
border: 1px solid lightgray;
25-
border-radius: 2px;
26-
box-shadow: 0 0 4px #dddddd;
21+
background-color: white;
22+
border: 1px solid lightgray;
23+
border-radius: 2px;
24+
box-shadow: 0 0 4px #dddddd;
2725

28-
cursor: pointer;
26+
cursor: pointer;
2927

30-
font-size: 16px;
28+
font-size: 16px;
3129

32-
align-items: center;
33-
display: flex;
34-
flex-direction: row;
30+
align-items: center;
31+
display: flex;
32+
flex-direction: row;
3533

36-
padding: 8px;
34+
padding: 8px;
3735
}
3836

39-
.slash-menu-item:hover, .slash-menu-item.selected {
40-
background-color: lightgray;
41-
}
37+
.slash-menu-item:hover,
38+
.slash-menu-item.selected {
39+
background-color: lightgray;
40+
}
Lines changed: 26 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,40 @@
11
.emoji-picker {
2-
z-index: 9999;
2+
background-color: white;
3+
border: 1px solid lightgray;
4+
border-radius: 2px;
5+
box-shadow: 0 0 8px #dddddd;
36

4-
background-color: white;
5-
border: 1px solid lightgray;
6-
border-radius: 2px;
7-
box-shadow: 0 0 8px #dddddd;
7+
display: grid;
8+
flex-direction: column;
9+
gap: 8px;
10+
height: fit-content;
11+
max-height: 100%;
812

9-
display: grid;
10-
flex-direction: column;
11-
gap: 8px;
12-
height: fit-content;
13-
max-height: 100%;
13+
overflow: auto;
1414

15-
overflow: auto;
15+
padding: 8px;
1616

17-
padding: 8px;
18-
19-
top: 8px;
17+
top: 8px;
2018
}
2119

2220
.emoji-picker-item {
23-
background-color: white;
24-
border: 1px solid lightgray;
25-
border-radius: 2px;
26-
box-shadow: 0 0 4px #dddddd;
21+
background-color: white;
22+
border: 1px solid lightgray;
23+
border-radius: 2px;
24+
box-shadow: 0 0 4px #dddddd;
2725

28-
cursor: pointer;
26+
cursor: pointer;
2927

30-
font-size: 16px;
28+
font-size: 16px;
3129

32-
align-items: center;
33-
display: flex;
34-
flex-direction: row;
30+
align-items: center;
31+
display: flex;
32+
flex-direction: row;
3533

36-
padding: 8px;
34+
padding: 8px;
3735
}
3836

39-
.emoji-picker-item:hover, .emoji-picker-item.selected {
40-
background-color: lightgray;
41-
}
37+
.emoji-picker-item:hover,
38+
.emoji-picker-item.selected {
39+
background-color: lightgray;
40+
}

examples/06-custom-schema/01-alert-block/Alert.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ export const Alert = createReactBlockSpec(
7171
return (
7272
<div className={"alert"} data-alert-type={props.block.props.type}>
7373
{/*Icon which opens a menu to choose the Alert type*/}
74-
<Menu withinPortal={false} zIndex={999999}>
74+
<Menu withinPortal={false}>
7575
<Menu.Target>
7676
<div className={"alert-icon-wrapper"} contentEditable={false}>
7777
<Icon

packages/mantine/src/popover/Popover.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,7 @@ export const Popover = (
1616
assertEmpty(rest);
1717

1818
return (
19-
<MantinePopover
20-
withinPortal={false}
21-
zIndex={10000}
22-
opened={opened}
23-
position={position}>
19+
<MantinePopover withinPortal={false} opened={opened} position={position}>
2420
{children}
2521
</MantinePopover>
2622
);

packages/shadcn/src/style.css

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,6 @@
7979
text-decoration: revert;
8080
}
8181

82-
.bn-shadcn [data-radix-popper-content-wrapper] {
83-
z-index: 99999 !important;
84-
}
85-
8682
.bn-shadcn .bn-editor:focus-visible {
8783
outline: none;
8884
}

0 commit comments

Comments
 (0)