From 8641059e0a12eee0b93aefe0ddaeb1c9a9248d69 Mon Sep 17 00:00:00 2001
From: Nick Lockhart <nicholas.o.lockhart@gmail.com>
Date: Thu, 17 Mar 2022 13:20:17 +1300
Subject: [PATCH] Move padding to `<summary>` element.

Otherwise the `<details>` element receives the click and doesn't trigger popover.

Co-authored-by: Lee Reilly <lee@github.com>
---
 src/components/giphy-toolbar-item.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/components/giphy-toolbar-item.js b/src/components/giphy-toolbar-item.js
index 19fafec..a5e7302 100644
--- a/src/components/giphy-toolbar-item.js
+++ b/src/components/giphy-toolbar-item.js
@@ -2,8 +2,12 @@
 import React from 'dom-chef'
 
 export default (
-  <details class="details-reset details-overlay toolbar-item select-menu select-menu-modal-right ghg-trigger btn-octicon p-2 p-md-1 mx-1">
-    <summary class="menu-target" aria-label="Insert a GIF" aria-haspopup="menu">
+  <details class="details-reset details-overlay toolbar-item select-menu select-menu-modal-right ghg-trigger btn-octicon">
+    <summary
+      class="menu-target p-2 p-md-1 mx-1"
+      aria-label="Insert a GIF"
+      aria-haspopup="menu"
+    >
       {'GIF'}
     </summary>
     <details-menu