From 31040e05bc0d7c337800493e4cc72f44cf19de7d Mon Sep 17 00:00:00 2001
From: samueljd <samuelj.ernest@gmail.com>
Date: Wed, 18 Jan 2023 12:26:29 +0530
Subject: [PATCH] fixed verse edit bleeding into verse number added blockable
 icons

---
 .../components/EditorPage/Scribex/Buttons.jsx | 36 +++++++++++++++----
 .../src/components/hooks/scribex/htmlmap.js   |  2 +-
 renderer/src/util/createVersificationUSFM.js  |  2 +-
 3 files changed, 32 insertions(+), 8 deletions(-)

diff --git a/renderer/src/components/EditorPage/Scribex/Buttons.jsx b/renderer/src/components/EditorPage/Scribex/Buttons.jsx
index 3bf70e65..d6d2873b 100644
--- a/renderer/src/components/EditorPage/Scribex/Buttons.jsx
+++ b/renderer/src/components/EditorPage/Scribex/Buttons.jsx
@@ -5,6 +5,11 @@ import {
   RectangleStackIcon,
   PencilIcon,
   ArrowDownOnSquareIcon,
+  Bars2Icon,
+  Bars4Icon,
+  ArrowUturnLeftIcon,
+  ArrowUturnRightIcon
+
 } from '@heroicons/react/24/outline';
 
 import { ArrowClockwise, ArrowCounterClockwise } from 'phosphor-react';
@@ -54,35 +59,54 @@ export default function Buttons(props) {
         )}
         aria-hidden="true"
         onClick={onSectionable}
+        title={sectionable ? "Expand all Chapters" : "Collapse Chapters"}
       />
 
       <PencilIcon
-        aria-label="Collection-Icon"
+        aria-label="Pencil-Icon"
         className={classNames(
           editable ? 'fill-current' : '',
           'h-5 mr-2 w-5 text-white cursor-pointer',
         )}
         aria-hidden="true"
         onClick={onEditable}
+        title={editable ? "Disable Edit" : "Enable Edit"}
       />
+      {blockable ? <Bars2Icon
+        aria-label="Article-Icon"
+        className='h-5 mr-2 w-5 text-white cursor-pointer'
+        aria-hidden="true"
+        onClick={onBlockable}
+        title="Collapse blocks"
+      /> :
+        <Bars4Icon
+          aria-label="List-Icon"
+          className='h-5 mr-2 w-5 text-white cursor-pointer'
+          aria-hidden="true"
+          onClick={onBlockable}
+          title="Split into blocks"
+        />}
 
-      <ArrowCounterClockwise
-        aria-label="Collection-Icon"
+      <ArrowUturnLeftIcon
+        aria-label="Undo-Icon"
         className="h-5 mr-2 w-5 text-white cursor-pointer"
         aria-hidden="true"
         onClick={() => undo()}
+        title={"Undo"}
       />
-      <ArrowClockwise
-        aria-label="Collection Icon"
+      <ArrowUturnRightIcon
+        aria-label="Redo-Icon"
         className="h-5 mr-2 w-5 text-white cursor-pointer"
         aria-hidden="true"
         onClick={() => redo()}
+        title="Redo"
       />
       <ArrowDownOnSquareIcon
-        aria-label="Save Icon"
+        aria-label="Save-Icon"
         className="h-5 mr-2 w-5 text-white cursor-pointer"
         aria-hidden="true"
         onClick={() => exportUsfm(bookCode)}
+        title="Save"
       />
     </>
   );
diff --git a/renderer/src/components/hooks/scribex/htmlmap.js b/renderer/src/components/hooks/scribex/htmlmap.js
index 41ec5378..902219ff 100644
--- a/renderer/src/components/hooks/scribex/htmlmap.js
+++ b/renderer/src/components/hooks/scribex/htmlmap.js
@@ -35,7 +35,7 @@ const htmlMap = (context) => ({
     verses: ({ atts }) => ({
       classList: ['mark', 'verse', `verse-${atts.number}`],
       id: `ch${context.lastChapter}v${atts.number}`,
-      tagName: 'a',
+      tagName: 'span',
     }),
   },
   graft: {
diff --git a/renderer/src/util/createVersificationUSFM.js b/renderer/src/util/createVersificationUSFM.js
index 36652a4c..5f51b5da 100644
--- a/renderer/src/util/createVersificationUSFM.js
+++ b/renderer/src/util/createVersificationUSFM.js
@@ -73,7 +73,7 @@ export const createVersificationUSFM = (
                   verses.push({
                     verseNumber: i.toString(),
                     verseText: '',
-                    contents: ['...'], // adding default text to verses to mitigate the empty verse bug
+                    contents: ['...'], // adding default text to verses to show the beginning of verse.
                   });
                 }
                 contents = contents.concat(verses);