From 4bbaa47f4aa083d0c0a48e77b8a107e22015521b Mon Sep 17 00:00:00 2001
From: GoOz <GoOz@users.noreply.github.com>
Date: Thu, 16 May 2019 10:20:10 +0200
Subject: [PATCH] Fix/minorfixes (#506)

* fix: Add field button margins

* fix: Remove field button margins

* fix: Edit form modal fullscreen on mobile
---
 .../ContactForm/ContactFormField.jsx          |  5 ++--
 .../__snapshots__/index.spec.jsx.snap         | 24 +++++++++----------
 src/components/Modals/ContactFormModal.jsx    |  1 +
 .../ContactFormModal.spec.jsx.snap            |  2 +-
 4 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/src/components/ContactCard/ContactForm/ContactFormField.jsx b/src/components/ContactCard/ContactForm/ContactFormField.jsx
index f62b75e03..9e771907a 100644
--- a/src/components/ContactCard/ContactForm/ContactFormField.jsx
+++ b/src/components/ContactCard/ContactForm/ContactFormField.jsx
@@ -49,10 +49,11 @@ class ContactFormField extends React.Component {
                 )}
                 <Button
                   icon="plus"
-                  subtle
+                  theme="text"
                   type="button"
                   onClick={() => this.addField(fields)}
                   label={t(`addLabel.${name}`)}
+                  className="u-ph-0 u-mh-0"
                 />
               </div>
             )}
@@ -89,7 +90,7 @@ class ContactFormField extends React.Component {
             round
             icon={'cross-small'}
             onClick={() => this.removeField(fields, index)}
-            className="contact-form__meta-button"
+            className="contact-form__meta-button u-m-0"
           />
         )}
       </div>
diff --git a/src/components/ContactCard/ContactForm/__snapshots__/index.spec.jsx.snap b/src/components/ContactCard/ContactForm/__snapshots__/index.spec.jsx.snap
index 4e1a01c50..c461ec521 100644
--- a/src/components/ContactCard/ContactForm/__snapshots__/index.spec.jsx.snap
+++ b/src/components/ContactCard/ContactForm/__snapshots__/index.spec.jsx.snap
@@ -169,7 +169,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -233,7 +233,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -260,7 +260,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
           </button>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
@@ -355,7 +355,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -419,7 +419,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -446,7 +446,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
           </button>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
@@ -541,7 +541,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -605,7 +605,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
             </div>
           </div>
           <button
-            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button"
+            className="styles__c-btn___-2Vnj styles__c-btn--secondary___3Br_N styles__c-btn--round___1Lkyl contact-form__meta-button u-m-0"
             onClick={[Function]}
             title="delete"
             type="button"
@@ -632,7 +632,7 @@ exports[`ContactForm should fill form with contact data 1`] = `
           </button>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
@@ -1041,7 +1041,7 @@ exports[`ContactForm should match snapshot 1`] = `
           </div>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
@@ -1123,7 +1123,7 @@ exports[`ContactForm should match snapshot 1`] = `
           </div>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
@@ -1205,7 +1205,7 @@ exports[`ContactForm should match snapshot 1`] = `
           </div>
         </div>
         <button
-          className="styles__c-btn___-2Vnj styles__c-btn--subtle___OknKf"
+          className="styles__c-btn___-2Vnj styles__c-btn--text___2Vp-2 u-ph-0 u-mh-0"
           onClick={[Function]}
           title={null}
           type="button"
diff --git a/src/components/Modals/ContactFormModal.jsx b/src/components/Modals/ContactFormModal.jsx
index 65a71c89e..a2f3d340d 100644
--- a/src/components/Modals/ContactFormModal.jsx
+++ b/src/components/Modals/ContactFormModal.jsx
@@ -25,6 +25,7 @@ const ContactFormModal = ({
   <Modal
     overflowHidden={true}
     dismissAction={onClose}
+    mobileFullscreen={true}
     into="body"
     size="xlarge"
   >
diff --git a/src/components/Modals/__snapshots__/ContactFormModal.spec.jsx.snap b/src/components/Modals/__snapshots__/ContactFormModal.spec.jsx.snap
index db78502ec..d71d743f5 100644
--- a/src/components/Modals/__snapshots__/ContactFormModal.spec.jsx.snap
+++ b/src/components/Modals/__snapshots__/ContactFormModal.spec.jsx.snap
@@ -5,7 +5,7 @@ exports[`ContactFormModal component should render a contact form in a modal 1`]
   closable={true}
   dismissAction={[MockFunction]}
   into="body"
-  mobileFullscreen={false}
+  mobileFullscreen={true}
   overflowHidden={true}
   primaryType="regular"
   secondaryType="secondary"