respond to comments

3DStreet · Jun 13, 2024 · dd4221f · dd4221f
1 parent d6cf2da
commit dd4221f
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 23 deletions.
diff --git a/src/editor/components/Main.js b/src/editor/components/Main.js
@@ -283,7 +283,7 @@ export default class Main extends Component {
           </div>
         )}
         {this.state.inspectorEnabled && (
-          <div id="scene-title">
+          <div id="action-bar">
             <ActionBar
               handleAddClick={this.toggleAddLayerPanel}
               isAddLayerPanelOpen={this.state.isAddLayerPanelOpen}

diff --git a/src/editor/components/components/ActionBar/ActionBar.component.jsx b/src/editor/components/components/ActionBar/ActionBar.component.jsx
@@ -15,14 +15,14 @@ const ActionBar = ({ handleAddClick, isAddLayerPanelOpen }) => {
   return (
     <div>
       {!isAddLayerPanelOpen && (
-        <div className={styles.wrapper} id="findme">
-          <button type="button" onClick={handleSelectClick} key="handButton">
+        <div className={styles.wrapper}>
+          <button type="button" onClick={handleSelectClick}>
             <ArrowDown24Icon />
           </button>
-          <button type="button" onClick={handleHandClick} key="handButton">
+          <button type="button" onClick={handleHandClick}>
             <ArrowDown24Icon />
           </button>
-          <button type="button" onClick={handleAddClick} key="addLayerButton">
+          <button type="button" onClick={handleAddClick}>
             <Circle20Icon />
           </button>
         </div>

diff --git a/src/editor/components/components/ActionBar/ActionBar.module.scss b/src/editor/components/components/ActionBar/ActionBar.module.scss
@@ -1,22 +1,20 @@
 @use '../../../style/variables.scss';
 
 .wrapper {
-    position: absolute;
-    background: rgba(55, 55, 55, 0.4);
-    transform: translateX(-50%);
-    bottom: 15%;
-    left: 50%;
-    border: unset;
-    height: 40px;
-    border-radius: 16px;
-    align-items: center;
-    display: flex;
-    column-gap: 8px;
-    padding: 8px 12px;
-    transition: 0.2s ease-in-out;
-    &:hover {
-      background: rgba(55, 55, 55, 0.6);
-    }
-
-  ;
+  position: absolute;
+  background: rgba(55, 55, 55, 0.4);
+  transform: translateX(-50%);
+  bottom: 15%;
+  left: 50%;
+  border: unset;
+  height: 40px;
+  border-radius: 16px;
+  align-items: center;
+  display: flex;
+  column-gap: 8px;
+  padding: 8px 12px;
+  transition: 0.2s ease-in-out;
+  &:hover {
+    background: rgba(55, 55, 55, 0.6);
+  }
 }
diff --git a/src/editor/index.js b/src/editor/index.js
@@ -192,6 +192,7 @@ Inspector.prototype = {
 
     Events.on('hidecursor', () => {
       this.cursor.pause();
+      this.selectEntity(null);
     });
     Events.on('showcursor', () => {
       this.cursor.play();