fix: not clickable when have notifications (#523)

Co-authored-by: Braian Mellor <[email protected]>
decentraland · Jul 15, 2024 · a968a67 · a968a67
1 parent 0f2aa5e
commit a968a67
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 18 deletions.
diff --git a/src/components/Notifications/Notifications.css b/src/components/Notifications/Notifications.css
@@ -6,19 +6,14 @@
   position: relative;
 }
 
-.dcl.notifications > div:first-child {
-  width: 32px;
-  height: 32px;
-}
-
 .dcl.notifications-bell {
   display: flex;
   background: none;
   border: none;
   padding: 0;
   margin: 0;
-  transition: filter 0.2s ease;
   cursor: pointer;
+  align-items: center;
   position: relative;
 }
 
@@ -31,8 +26,7 @@
   width: 16px;
   height: 16px;
   position: relative;
-  top: -24px;
-  left: 19px;
+  left: -13px;
   cursor: pointer;
 }
 

diff --git a/src/components/Notifications/Notifications.tsx b/src/components/Notifications/Notifications.tsx
@@ -54,21 +54,19 @@ export default function Notifications({
 
   return (
     <div className="dcl notifications">
-      <div>
-        <button
-          className={classNames('dcl notifications-bell', {
-            'notifications-active': newNotificationsCount > 0
-          })}
-          onClick={onClick}
-        >
-          {!isOpen ? <NotificationBell /> : <NotificationBellActive />}
-        </button>
+      <button
+        className={classNames('dcl notifications-bell', {
+          'notifications-active': newNotificationsCount > 0
+        })}
+        onClick={onClick}
+      >
+        {!isOpen ? <NotificationBell /> : <NotificationBellActive />}
         {!isOpen && newNotificationsCount > 0 && (
           <div className="dcl notifications-counter">
             <Counter count={newNotificationsCount} />
           </div>
         )}
-      </div>
+      </button>
       {isOpen && (
         <NotificationsFeed
           isOpen={isOpen}