-
Notifications
You must be signed in to change notification settings - Fork 118
Fix - Actionbar menus are hidden behind the document viewer #2048
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
Conversation
@@ -32,7 +32,6 @@ | |||
justify-content: center; | |||
font-size: 14px; | |||
flex-direction: column; | |||
z-index: 100; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The .gn-main-event-container
is used by multiple components including global loaders and feeback messages so I would prefer to not change this.
The other strange thins is the used of .gn-main-event-container
inside the MetadataPreview component.
I would prefer we have a dedicated className and use of utility classes:
import FlexBox from '@mapstore/framework/components/layout/FlexBox';
import React from 'react';
function MetadataPreview({
url
}) {
return (
<FlexBox column centerChildren classNames={['gn-metadata-preview', '_corner-tl', '_absolute', '_fill']}>
<iframe
frameBorder="0"
key={url}
src={url}
style={{
width: '100%',
height: '100%'
}} />
</FlexBox>
);
}
please check if this is working as expected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine. Thanks!
Also fixed a couple of issues when adhoc testing
- Hide footer placeholder in viewer page
- download menu button alignment
…bar_style * commit '73a073ef6d312baaf10991cc2f4b5f17c5e40b8e': Resource page config based on type (GeoNode#2050) [create-pull-request] automated change (GeoNode#2066) Fix - Actionbar menus are hidden behind the document viewer (GeoNode#2048) [create-pull-request] automated change (GeoNode#2065) Updated MS submodule to latest stable (GeoNode#2051) [create-pull-request] automated change (GeoNode#2064) Enhance close open right panel handler (GeoNode#2059) [create-pull-request] automated change (GeoNode#2063) Update style hovering user icon (GeoNode#2060) [create-pull-request] automated change (GeoNode#2062) GeoNode#2020 - Update edit menu of the dataset and layer settings (GeoNode#2026) [create-pull-request] automated change (GeoNode#2061) Remove thumbnail from a resource (GeoNode#2058) Fixing Favorites URL on the user menu (GeoNode#2053) fix tutorial theme (GeoNode#2049) [create-pull-request] automated change (GeoNode#2046) Restore primary color (GeoNode#2044) [create-pull-request] automated change (GeoNode#2045) Implementation of redirect system (GeoNode#2038) [create-pull-request] automated change (GeoNode#2043)
…viewer * commit '73a073ef6d312baaf10991cc2f4b5f17c5e40b8e': Resource page config based on type (GeoNode#2050) [create-pull-request] automated change (GeoNode#2066) Fix - Actionbar menus are hidden behind the document viewer (GeoNode#2048) [create-pull-request] automated change (GeoNode#2065) Updated MS submodule to latest stable (GeoNode#2051) [create-pull-request] automated change (GeoNode#2064) Enhance close open right panel handler (GeoNode#2059)
Description
This PR fixes the menu items hidden in viewer
Screenshot