-
Notifications
You must be signed in to change notification settings - Fork 403
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
[Backport 2022.01.xx-georchestra] #8086 Layout changes and other improvements (#8085) #8175
Merged
tdipisa
merged 2 commits into
geosolutions-it:2022.01.xx-georchestra
from
alexander-fedorenko:backport-2021.02.xx-georchestra-8086
May 9, 2022
Merged
[Backport 2022.01.xx-georchestra] #8086 Layout changes and other improvements (#8085) #8175
tdipisa
merged 2 commits into
geosolutions-it:2022.01.xx-georchestra
from
alexander-fedorenko:backport-2021.02.xx-georchestra-8086
May 9, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ons-it#8085) * # 150 Layout changes of the UI - Moving annotations to the left sidebar (TOC), adding annotation management tools into its toolbar. * Updated test to be align with the changed position of annotations panel * Changes for print plugin required to support printing of additional layers. * Minor customizations for "Home" and "Login" plugins to make them properly styled both for Omnibar and SidebarMenu Adding abstract element for sidebar menu that could be used as a tool. Making OmniBar disappear on map viewer page if sidebarMenu was instantiated. * Update of the rest plugins that were presented in BurgerMenu to appear in SidebarMenu * rollback, was causing problems when some glyphicons were not rendered * Corrections of style for legacy burger menu * - Hide burger menu when SidebarMenu is active, keep OmniBar visible as it is a container for search plugin - Updated configuration for right panels: aligned their width (all to 550px), connected them to the layout state so that they track for right/bottom paddings. - Provided different way to apply paddings for dock panels and for toolbars (omnibar, map toolbar) - New component to wrap dock panels. - Style changes * Skip attempt to sum-up dockSize as percentage amount. * Moved layout-based style from omnibar to search plugin to prevent infinite loop * Added toggle support for part of the tools; Style fixes. * Selector & reducer update * Sidebar menu with tracking of the window height and hiding elements into dropdown dynamically. * - Sidebar - keep last active item tracked. - Burger menu, fix for visibility condition - Minor fix to the selector to get paddings for docks and toolbars depending on open tools. Definitely not the most beauty way to resolve the issue, but it helps to prevent gazzilions of unneeded re-renders for components when store state changes by ANY action. - All menu elements are togglable. - Search bar with new behavior (- media queries, + checks if width fit to the screen) - Fix for react-container-dimensions & dock components by updating it to the latest version (only used by docks, so it's easy to test/safe to update) - Slightly changes styles of preview plugin to make it look fine with the sidebar menu. * Prevent reset of last active item from additional list when user toggles item in the main list * Support of having plugins list that define dock panel that needs to be open exclusively in sidebar. All other panels from the list will be deactivated automatically. Configurable/extendible via "miscSettings.excludsiveDockPanels" in localConfig.json * - Updated annotations panels order (since it's moved from right to left) - Deactivate annotations when measure is on and vice versa - Toggle off any open panel in right sidebar when feature editor is open - Toggle off Identify panel when any right panel is getting activated. - Z-index readjustments * - Fixes for searchbar when omnibar contains elements; - Add cadastrapp to the list of exclusive panels - Re-checking that sidebar is properly instantiated on its update. Sometimes onInit isn't running on component mount. - Making user extensions plugin close feature editor * Locale strings for sidebar menu * Fixes search to be styled consistently with or without sidebar menu. Correction of elements order in omnibar on home page. * Fixes search to be styled consistently with or without sidebar menu. * Amending existing tests to make them work after layout changes; Minor fixes to the epic to keep old logic for "bottom" offset. * Basic test coverage for implemented functionality * Reverting flex styles for omnibar. Using simpler approach for proper searchbar positioning. * Fix of the bug that prevents prioritized container to be properly selected if more than one container is defined as a function in plugin. * - Changes in plugin priorities to make them appear in BurgerMenu by default. - SLight update to the burger menu to make it's possible to track whether it's mounted on the page or not. - Resolving issue with Home and Login plugins to be rendered in sidebarMenu IF there is no BurgerMenu, otherwise they should be rendered in omnibar; - * Reducing button sizes; Adding shadow to the sidebar; Replaced save tool icon. * Making option to print additional layers optional and disabled by default, updating option name. * Removed monitored state. * Removed BurgerMenu from desktop plugins. * Share plugin tests fix. * Order matters * 15/30px size for medium-size buttons * - Separate panel component with support of tracking of container width - Fix for measure plugin; - Fix of map grid element height on home page. * Fix for annotations plugin tests * Revert of hardcoded toolbar buttons in TOC for annotations. Defining buttons in Annotations plugin instead. * Correction to make cards size approximately same as it was before button sizes change * Using common button size in less stylesheet of SidebarMenu; Migration guide. * - Tracking of the case when left/right panels are open via state. - Style fix for mapstore logo, spinner; - Widgets plugin corrections, removed hardcoded offset, using offset provided by state. - Amended tests * Updated migration documentation with changes to the pluginsConfig.json * - Map export, Save, SaveAs plugins - making them work again by adding doNotHide flag into container configuration. * DockContainer and ResponsivePanel documentation * Added card-height variable. Applied it to the min-height of the sidegrid cards & TOC layers list * Missed lines * Missed lines /2 * Use card-height in TOC styles * Suggested change to the documentation with minor fix * Update web/client/configs/pluginsConfig.json Co-authored-by: Lorenzo Natali <[email protected]> * Properly hide overlay layers printing options by default (it is enabled if opposite is not passed by props) * Properly hide overlay layers printing options by default (it is enabled if opposite is not passed by props) * Missing doNotHide for DeleteMap plugin Co-authored-by: Lorenzo Natali <[email protected]> (cherry picked from commit 8d438d9)
* Context creator - map configuration - use SidebarMenu instead of burger menu. Properly refresh sidebar menu activity state on map reload * Class name should be applied to the container as it's important for proper styling of catalog in modal map viewer in geostory * Custom compass offset on Cesium viewer in case if sidebar is active. Passing sidebar state to the map plugin. * Get back original containers, applying same priority to the containers and hiding menu item in sidebar if burger menu exists on the page Rolling back updatableLayersCount selector item. * Revert "Custom compass offset on Cesium viewer in case if sidebar is active. Passing sidebar state to the map plugin." This reverts commit 704c205. * Static position update of compass * Update of the pluginsFilterOverride to make it works with a custom list of plugins passed by props (cherry picked from commit 29cbb12)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[Backport 2022.01.xx-georchestra]
#8086 Layout changes and other improvements (#8085)
#8086 Bugfixes for layout changes (#8169)