Skip to content

Commit

Permalink
geosolutions-it#4079 units dropdown is not fully visible in measure t…
Browse files Browse the repository at this point in the history
…ool (geosolutions-it#4081)

It turned out that the unit dropdown options in measure dialog gets
hidden behind the dialog and prevent user to select. This commit
fix and make the dropdown options visible and selectable.

Note:
This commits also change the border layout by allowing it to accept
style to override the default border inline style.
  • Loading branch information
kasongoyo authored and offtherailz committed Aug 9, 2019
1 parent 58c7e4f commit 365875c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 3 additions & 2 deletions web/client/components/layout/BorderLayout.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,14 @@ const React = require('react');
* /></BorderLayout>
*
*/
module.exports = ({id, children, header, footer, columns, height, className, bodyClassName = "ms2-border-layout-body"}) =>
module.exports = ({id, children, header, footer, columns, height, style={}, className, bodyClassName = "ms2-border-layout-body"}) =>
(<div id={id} className={className} style={{
display: "flex",
flexDirection: "column",
width: "100%",
height: "100%",
overflow: "hidden"
overflow: "hidden",
...style
}}>
{header}
<div className={bodyClassName} style={{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -256,6 +256,7 @@ class MeasureComponent extends React.Component {
return (
<BorderLayout
id={this.props.id}
style={{overflow: 'visible'}}
header={
<div>
<ButtonToolbar style={{width: '100%', marginBottom: 15, marginTop: 8, display: 'flex', justifyContent: 'center'}}>
Expand Down

0 comments on commit 365875c

Please sign in to comment.