merge: pull request #629 from saif-ellafi/dont-select-unselectable-layers

Do not select unselectable layers

Author: hm21

CHANGELOG.md

@@ -1,7 +1,10 @@
 # Changelog
 
+## 11.2.3
+- **FIX**(multiselect): Resolve issue where layers could still be selected even when `enableSelection` for the layer was set to `false`. This resolves issue [#628](https://github.com/hm21/pro_image_editor/issues/628).
+
 ## 11.2.2
-- **FIX**(main-editor): Resolved an issue where the `replaceLayer` function broke the logic that ensured layers resized correctly when the screen size changed. This resolves issue [#624](https://github.com/hm21/pro_image_editor/issues/624) and issue [#626](https://github.com/hm21/pro_image_editor/issues/626).
+- **FIX**(main-editor): Resolve issue where the `replaceLayer` function broke the logic that ensured layers resized correctly when the screen size changed. This resolves issue [#624](https://github.com/hm21/pro_image_editor/issues/624) and issue [#626](https://github.com/hm21/pro_image_editor/issues/626).
 
 ## 11.2.1
 - **FIX**(state-history): Resolve issue where updating the background-image overwrote previous states.

lib/features/main_editor/main_editor.dart

@@ -819,7 +819,7 @@ class ProImageEditorState extends State<ProImageEditor>
     if (removeLayerIndex >= 0) {
       activeLayers.removeAt(removeLayerIndex);
     }
-    if (!blockSelectLayer) {
+    if (!blockSelectLayer && layer.interaction.enableSelection) {
       layerInteractionManager.addSelectedLayer(layer.id);
     }
     _checkInteractiveViewer();
@@ -2359,8 +2359,13 @@ class ProImageEditorState extends State<ProImageEditor>
   /// active layer.
   Layer? selectLayerById(String id, {bool enableMultiSelect = false}) {
     int index = activeLayers.indexWhere((layer) => layer.id == id);
+    if (index == -1) return null;
+
     Layer? layer = activeLayers[index];
 
+    // Check if the layer allows selection
+    if (!layer.interaction.enableSelection) return null;
+
     if (!enableMultiSelect) layerInteractionManager.clearSelectedLayers();
 
     layerInteractionManager.addSelectedLayer(id);
@@ -2371,7 +2376,9 @@ class ProImageEditorState extends State<ProImageEditor>
   /// Selects all available layers.
   void selectAllLayers() {
     layerInteractionManager.setSelectedLayers(
-      activeLayers.map((layer) => layer.id),
+      activeLayers
+          .where((layer) => layer.interaction.enableSelection)
+          .map((layer) => layer.id),
     );
     _controllers.uiLayerCtrl.add(null);
   }

lib/features/main_editor/services/layer_drag_selection_service.dart

@@ -151,6 +151,9 @@ class LayerDragSelectionService {
 
       if (size == null || size.isEmpty) continue;
 
+      // Skip layers that have enableSelection set to false
+      if (!layer.interaction.enableSelection) continue;
+
       Offset fractionalOffset = layer.isTextLayer
           ? textOffset
           : layer.isPaintLayer

lib/features/main_editor/services/main_editor_layers_service.dart

@@ -230,7 +230,8 @@ class MainEditorLayersService {
     if (layer.groupId == null) return;
     // If layer is part of a group, handle group selection
     Set<String> groupIds = _activeLayers
-        .where((l) => l.groupId == layer.groupId)
+        .where(
+            (l) => l.groupId == layer.groupId && l.interaction.enableSelection)
         .map((l) => l.id)
         .toSet();
 

pubspec.yaml

@@ -1,6 +1,6 @@
 name: pro_image_editor
 description: "A Flutter image editor: Seamlessly enhance your images with user-friendly editing features."
-version: 11.2.2
+version: 11.2.3
 homepage: https://github.com/hm21/pro_image_editor/
 repository: https://github.com/hm21/pro_image_editor/
 documentation: https://github.com/hm21/pro_image_editor/