From 536cd22988407ab86099595daa4ffe4f054b2914 Mon Sep 17 00:00:00 2001
From: bobzel <zzzman@gmail.com>
Date: Wed, 7 Feb 2024 22:43:00 -0500
Subject: fixed icon placement in linkMenuItem.

---
 src/client/views/linking/LinkMenuItem.scss | 32 ++++++++++++++----------------
 src/client/views/linking/LinkMenuItem.tsx  | 14 ++++++-------
 2 files changed, 22 insertions(+), 24 deletions(-)

(limited to 'src/client/views/linking')

diff --git a/src/client/views/linking/LinkMenuItem.scss b/src/client/views/linking/LinkMenuItem.scss
index 44c74236f..66ddd6eca 100644
--- a/src/client/views/linking/LinkMenuItem.scss
+++ b/src/client/views/linking/LinkMenuItem.scss
@@ -18,6 +18,21 @@
         align-items: center;
         display: flex;
 
+        .linkMenu-icon-wrapper {
+            //border: 0.5px solid rgb(161, 161, 161);
+            margin-right: 2px;
+            padding-right: 2px;
+
+            .linkMenu-icon {
+                float: left;
+                width: 12px;
+                height: 12px;
+                padding: 1px;
+                margin-right: 3px;
+                // color: rgb(161, 161, 161);
+            }
+        }
+
         .linkMenu-text {
             width: 100%;
 
@@ -34,22 +49,6 @@
                 display: flex;
                 align-items: center;
                 min-height: 20px;
-
-                .destination-icon-wrapper {
-                    //border: 0.5px solid rgb(161, 161, 161);
-                    margin-right: 2px;
-                    padding-right: 2px;
-
-                    .destination-icon {
-                        float: left;
-                        width: 12px;
-                        height: 12px;
-                        padding: 1px;
-                        margin-right: 3px;
-                        // color: rgb(161, 161, 161);
-                    }
-                }
-
                 .linkMenu-destination-title {
                     text-decoration: none;
                     font-size: 13px;
@@ -111,7 +110,6 @@
     //@extend: right;
     position: relative;
     display: flex;
-    opacity: 0;
 
     .linkMenu-deleteButton {
         width: 20px;
diff --git a/src/client/views/linking/LinkMenuItem.tsx b/src/client/views/linking/LinkMenuItem.tsx
index a44ad26cd..e694806a5 100644
--- a/src/client/views/linking/LinkMenuItem.tsx
+++ b/src/client/views/linking/LinkMenuItem.tsx
@@ -178,8 +178,13 @@ export class LinkMenuItem extends ObservableReactComponent<LinkMenuItemProps> {
                         onPointerDown={this.onLinkButtonDown}>
                         <div className="linkMenu-item-buttons">
                             <Tooltip title={<div className="dash-tooltip">Edit Link</div>}>
-                                <div className="button" ref={this._editRef} onPointerDown={this.onEdit} onClick={e => e.stopPropagation()}>
-                                    <FontAwesomeIcon className="fa-icon" icon="edit" size="sm" />
+                                <div className="linkMenu-icon-wrapper" ref={this._editRef} onPointerDown={this.onEdit} onClick={e => e.stopPropagation()}>
+                                    <FontAwesomeIcon className="linkMenu-icon" icon="edit" size="sm" />
+                                </div>
+                            </Tooltip>
+                            <Tooltip title={<div className="dash-tooltip">Show/Hide Link</div>}>
+                                <div title="click to show link" className="linkMenu-icon-wrapper" onPointerDown={this.onIconDown}>
+                                    <FontAwesomeIcon className="linkMenu-icon" icon={destinationIcon} size="sm" />
                                 </div>
                             </Tooltip>
                         </div>
@@ -206,11 +211,6 @@ export class LinkMenuItem extends ObservableReactComponent<LinkMenuItemProps> {
                                 </p>
                             ) : null}
                             <div className="linkMenu-title-wrapper">
-                                <Tooltip title={<div className="dash-tooltip">Show/Hide Link</div>}>
-                                    <div title="click to show link" className="destination-icon-wrapper" onPointerDown={this.onIconDown}>
-                                        <FontAwesomeIcon className="destination-icon" icon={destinationIcon} size="sm" />
-                                    </div>
-                                </Tooltip>
                                 <Tooltip title={<div className="dash-tooltip">Follow Link</div>}>
                                     <p className="linkMenu-destination-title">
                                         {this._props.linkDoc.linksToAnnotation && Cast(this._props.destinationDoc.data, WebField)?.url.href === this._props.linkDoc.annotationUri ? 'Annotation in' : ''} {StrCast(title)}
-- 
cgit v1.2.3-70-g09d2