Skip to content

Commit

Permalink
geosolutions-it#10424: The measure tool is not opened properly if Mea…
Browse files Browse the repository at this point in the history
…sure plugin has 'showCoordinateEditor' with true into cfg.defaultOptions

Description:
- resolve review comments [change the added class name + handle all mentioned cases]
- add unit test
  • Loading branch information
mahmoudadel54 committed Jun 20, 2024
1 parent bf22c74 commit 7a23713
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion web/client/plugins/Measure.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ const MeasurePlugin = connect(
}))
)(({coordsAeronauticalEnabled, ...props}) => {
return (
<div className={props.mapType !== MapLibraries.CESIUM ? "measure-container" : "measure-container measure-cesium"} style={{top: coordsAeronauticalEnabled ? 88 : 48}}>
<div className={props.mapType !== MapLibraries.CESIUM && props?.defaultOptions?.showCoordinateEditor ? "measure-container measure-coords-editor-2d" : "measure-container"} style={{top: coordsAeronauticalEnabled ? 88 : 48}}>
{
props.mapType === MapLibraries.CESIUM
? <div id="measure-cesium-wrapper"/>
Expand Down
2 changes: 1 addition & 1 deletion web/client/themes/default/less/measure.less
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@
}
}
}
.measure-container.measure-cesium{
.measure-container:not(.measure-coords-editor-2d){
position: absolute;
z-index: 100;
right: 46px;
Expand Down

0 comments on commit 7a23713

Please sign in to comment.