#4349 - Macro: Selection does not work in sequence editing with Shift…

…+Up/Down arrow combination
epam · Apr 22, 2024 · cb89c5d · cb89c5d
1 parent d474ea2
commit cb89c5d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/packages/ketcher-core/src/application/editor/modes/SequenceMode.ts b/packages/ketcher-core/src/application/editor/modes/SequenceMode.ts
@@ -225,10 +225,10 @@ export class SequenceMode extends BaseMode {
   public mousedown(event: MouseEvent) {
     const eventData = event.target?.__data__;
     const isEventOnSequenceItem = eventData instanceof BaseSequenceItemRenderer;
-    console.log(this.isEditMode);
-
-    if (this.isEditMode && isEventOnSequenceItem && !event.shiftKey) {
+    if (this.isEditMode) {
       SequenceRenderer.resetLastCaretPosition();
+    }
+    if (this.isEditMode && isEventOnSequenceItem && !event.shiftKey) {
       SequenceRenderer.setCaretPositionBySequenceItemRenderer(
         eventData as BaseSequenceItemRenderer,
       );

diff --git a/packages/ketcher-core/src/application/render/renderers/sequence/SequenceRenderer.ts b/packages/ketcher-core/src/application/render/renderers/sequence/SequenceRenderer.ts
@@ -333,10 +333,10 @@ export class SequenceRenderer {
 
     for (let i = 0; i < finalArray.length; i++) {
       const subArray = finalArray[i];
-      if (subArray.length > 30) {
+      if (subArray.length > this.NUMBER_OF_SYMBOLS_IN_ROW) {
         const newSubArrays: Array<SubChainNode[]> = [];
         while (subArray.length > 0) {
-          newSubArrays.push(subArray.splice(0, 30));
+          newSubArrays.push(subArray.splice(0, this.NUMBER_OF_SYMBOLS_IN_ROW));
         }
         finalArray.splice(i, 1, ...newSubArrays);
         i += newSubArrays.length - 1;