Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up References Dialog edit modal styling #114

Closed
olafgrabienski opened this issue Feb 26, 2024 · 3 comments · Fixed by #150
Closed

Clean up References Dialog edit modal styling #114

olafgrabienski opened this issue Feb 26, 2024 · 3 comments · Fixed by #150
Labels
question Further information is requested

Comments

@olafgrabienski
Copy link
Member

In the content edit form, when I add or edit another content item using References Dialog, the sidebar panel on the primary content edit form is rendered above the References Dialog overlay, making it unclear to which form the sidebar belongs (see screenshots below).

Changing z-index for the sidebar from 103 to 100 fixes the issue for me:

.gin--edit-form .edit-form--sidebar.page-wrapper .layout-region-node-secondary {
  z-index: 100;
}

(Some parts of commit 254bdb7 look related.)

Expected behavior:
image

Actual behavior:
image

@laryn
Copy link
Member

laryn commented Mar 25, 2024

@olafgrabienski Is this sidebar z-index still an issue with the most recent code? It does look like References Dialog needs some cleanup as well, in terms of dialogs (size, placement, etc.)

Other notes about References Dialog:

  • Maybe we need to make sure the "sidebar edit details" don't apply within modals, it's messing the bottom of the modal edit display
  • There are some icons in use with RD that should be adjusted to use SVG and make sure they work with dark mode
    CleanShot 2024-03-25 at 16 34 56@2x
  • RD should make modals display better in terms of size and placement (they display poorly with Seven as well)

@laryn laryn added the question Further information is requested label Mar 27, 2024
@olafgrabienski
Copy link
Member Author

Just tested: with the current code the issue is fixed for me, great!

And I agree, the display of References Dialog could need some love.

Maybe we need to make sure the "sidebar edit details" don't apply within modals (...)

Are you referring to the vertical tabs (Publishing actions, Authoring information, ...)? Those should stay, in my opinion, but I'm sure they can be styled in a more pleasant way.

@laryn laryn changed the title Sidebar is above References Dialog overlay Clean up References Dialog edit modal styling Apr 2, 2024
laryn added a commit that referenced this issue Jul 4, 2024
@laryn
Copy link
Member

laryn commented Sep 14, 2024

Closing for now as things are looking a lot better.
CleanShot 2024-09-13 at 23 26 07@2x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants