Skip to content

Commit

Permalink
changed z-index for overlay to show over sidebar
Browse files Browse the repository at this point in the history
  • Loading branch information
ajayadav09 committed Feb 19, 2024
1 parent 7c0690d commit 3cf84ab
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ html.nfd-interface-interface-skeleton__html-container {
}

&__overlay {

// z-index: to display over sidebar.
&--disabled {
position: fixed;
top: 0;
Expand All @@ -49,7 +49,7 @@ html.nfd-interface-interface-skeleton__html-container {
bottom: 0;
background: rgba(var(--nfd-onboarding-secondary-rgb), 0.3);
cursor: not-allowed;
z-index: 30;
z-index: 1000001;
}
}
}
Expand Down
4 changes: 2 additions & 2 deletions src/OnboardingSPA/steps/SiteGen/Editor/Header/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ const StepSiteGenEditorHeader = () => {
}, [ currentData ] );

useEffect( () => {
if ( isSaving ) {
if ( isSaving || isRegenerating ) {
setInteractionDisabled( true );
} else {
setInteractionDisabled( false );
Expand All @@ -233,7 +233,7 @@ const StepSiteGenEditorHeader = () => {
return () => {
setInteractionDisabled( false );
};
}, [ isSaving ] );
}, [ isSaving, isRegenerating ] );

Check warning on line 236 in src/OnboardingSPA/steps/SiteGen/Editor/Header/index.js

View workflow job for this annotation

GitHub Actions / Run Lint Checks

React Hook useEffect has a missing dependency: 'setInteractionDisabled'. Either include it or remove the dependency array

return (
<>
Expand Down

0 comments on commit 3cf84ab

Please sign in to comment.