Disable opacity for points

xcube-dev · Jul 19, 2024 · aec6ecd · aec6ecd
1 parent 7699fcc
commit aec6ecd
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 13 deletions.
diff --git a/src/components/PlaceSelect.tsx b/src/components/PlaceSelect.tsx
@@ -98,8 +98,6 @@ export default function PlaceSelect({
   selectedPlaceId = selectedPlaceId || "";
   selectedPlaceGroupIds = selectedPlaceGroupIds || [];
 
-  console.log("selectedPlaceInfo:", selectedPlaceInfo);
-
   const selectedPlaceGroupId =
     selectedPlaceGroupIds!.length === 1 ? selectedPlaceGroupIds![0] : null;
 
@@ -146,7 +144,15 @@ export default function PlaceSelect({
     selectedPlaceGroupId.startsWith(USER_ID_PREFIX) &&
     selectedPlaceId !== "";
 
-  let actions;
+  let actions = [
+    <ToolButton
+      key="locatePlace"
+      onClick={locateSelectedPlace}
+      tooltipText={i18n.get("Locate place in map")}
+      icon={<TravelExploreIcon />}
+    />,
+  ];
+
   if (!editMode && isEditableUserPlace) {
     const handleEditButtonClick = () => {
       setEditMode(true);
@@ -179,13 +185,7 @@ export default function PlaceSelect({
         tooltipText={i18n.get("Remove place")}
         icon={<RemoveCircleOutlineIcon />}
       />,
-      <ToolButton
-        key="locatePlace"
-        onClick={locateSelectedPlace}
-        tooltipText={i18n.get("Locate place in map")}
-        icon={<TravelExploreIcon />}
-      />,
-    ];
+    ].concat(actions);
   }
 
   return (
@@ -204,6 +204,7 @@ export default function PlaceSelect({
         <PlaceStyleEditor
           anchorEl={styleAnchorEl}
           setAnchorEl={setStyleAnchorEl}
+          isPoint={selectedPlaceInfo.place.geometry.type === "Point"}
           placeStyle={selectedPlaceInfo}
           updatePlaceStyle={updatePlaceStyle}
         />

diff --git a/src/components/PlaceStyleEditor/PlaceStyleEditor.tsx b/src/components/PlaceStyleEditor/PlaceStyleEditor.tsx
@@ -72,19 +72,21 @@ const styles = makeStyles({
   colorMenuItemBox: { width: "104px", height: "18px" },
 });
 
-interface FeatureStyleEditorProps extends WithLocale {
+interface PlaceStyleEditorProps extends WithLocale {
   anchorEl: HTMLElement | null;
   setAnchorEl: (anchorEl: HTMLElement | null) => void;
+  isPoint: boolean;
   placeStyle: PlaceStyle;
   updatePlaceStyle: (placeStyle: PlaceStyle) => void;
 }
 
 const PlaceStyleEditor = ({
   anchorEl,
   setAnchorEl,
+  isPoint,
   placeStyle,
   updatePlaceStyle,
-}: FeatureStyleEditorProps) => {
+}: PlaceStyleEditorProps) => {
   const [menuAnchorEl, setMenuAnchorEl] = useState<HTMLElement | null>(null);
 
   function handleColorClick(event: MouseEvent<HTMLDivElement>) {
@@ -102,7 +104,10 @@ const PlaceStyleEditor = ({
       >
         <Box sx={styles.container}>
           <Typography sx={styles.colorLabel}>{i18n.get("Color")}</Typography>
-          <Typography sx={styles.opacityLabel}>
+          <Typography
+            sx={styles.opacityLabel}
+            color={isPoint ? "text.secondary" : "text.primary"}
+          >
             {i18n.get("Opacity")}
           </Typography>
           <Box
@@ -112,6 +117,7 @@ const PlaceStyleEditor = ({
           />
           <Slider
             sx={styles.opacityValue}
+            disabled={isPoint}
             size="small"
             min={0}
             max={1}