@@ -737,13 +737,14 @@ function toggleAddRiver() {
737737 const pressed = ensureEl ( "addRiver" ) . classList . contains ( "pressed" ) ;
738738 if ( pressed ) {
739739 unpressClickToAddButton ( ) ;
740- ensureEl ( "addNewRiver" ) . classList . remove ( "pressed" ) ;
740+ findEl ( "addNewRiver" ) ? .classList . remove ( "pressed" ) ;
741741 return ;
742742 }
743743
744744 addFeature . querySelectorAll ( "button.pressed" ) . forEach ( b => b . classList . remove ( "pressed" ) ) ;
745745 addRiver . classList . add ( "pressed" ) ;
746- ensureEl ( "addNewRiver" ) . classList . add ( "pressed" ) ;
746+ if ( ! document . getElementById ( "addNewRiver" ) ) window . Controllers . RiversOverview . open ( ) ;
747+ findEl ( "addNewRiver" ) ?. classList . add ( "pressed" ) ;
747748 closeDialogs ( ".stable" ) ;
748749 viewbox . style ( "cursor" , "crosshair" ) . on ( "click" , addRiverOnClick ) ;
749750 tip ( "Click on map to place new river or extend an existing one. Hold Shift to place multiple rivers" , true , "warn" ) ;
@@ -891,13 +892,13 @@ function addRiverOnClick() {
891892 if ( d3 . event . shiftKey === false ) {
892893 Lakes . cleanupLakeData ( ) ;
893894 unpressClickToAddButton ( ) ;
894- ensureEl ( "addNewRiver" ) . classList . remove ( "pressed" ) ;
895- if ( addNewRiver . offsetParent ) riversOverviewRefresh . click ( ) ;
895+ findEl ( "addNewRiver" ) ? .classList . remove ( "pressed" ) ;
896+ if ( findEl ( " addNewRiver" ) ? .offsetParent ) riversOverviewRefresh . click ( ) ;
896897 }
897898}
898899
899900function toggleAddMarker ( ) {
900- const pressed = ensureEl ( "addMarker" ) . classList . contains ( "pressed" ) ;
901+ const pressed = findEl ( "addMarker" ) ? .classList . contains ( "pressed" ) ;
901902 if ( pressed ) {
902903 unpressClickToAddButton ( ) ;
903904 return ;
0 commit comments