Merge pull request #142 from bosonprotocol/tagline-commit-button-mand…

…atory refactor: tagline is mandatory in commit button
bosonprotocol · Apr 17, 2024 · 35ba87d · 35ba87d
2 parents a8f093b + 2f5ccc7
commit 35ba87d
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 20 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -3,7 +3,7 @@
   "version": "0.1.0",
   "private": true,
   "dependencies": {
-    "@bosonprotocol/react-kit": "^0.29.1",
+    "@bosonprotocol/react-kit": "^0.30.0-alpha.1",
     "@krakenjs/zoid": "^10.3.3",
     "@svgr/webpack": "^8.1.0",
     "@testing-library/jest-dom": "^5.16.5",

diff --git a/public/scripts/commit-button-example.html b/public/scripts/commit-button-example.html
@@ -27,7 +27,6 @@ <h3>Commit button example</h3>
       modalMargin: "2%",
       lookAndFeel: "modal",
       disabled: false,
-      tagline: true,
       buttonStyle: {
         minWidth: "100px",
         minHeight: "200px",

diff --git a/src/components/widgets/commitButton/CommitButton.tsx b/src/components/widgets/commitButton/CommitButton.tsx
@@ -104,20 +104,11 @@ export function CommitButton() {
             "iframe"
           );
         }}
-        {...(props.tagline
-          ? {
-              tagline: true,
-              onTaglineClick: () => {
-                PurchaseOverviewModal({
-                  modalMargin
-                }).renderTo(
-                  window.parent,
-                  renderToSelector || "body",
-                  "iframe"
-                );
-              }
-            }
-          : { tagline: false })}
+        onTaglineClick={() => {
+          PurchaseOverviewModal({
+            modalMargin
+          }).renderTo(window.parent, renderToSelector || "body", "iframe");
+        }}
       />
     </>
   );