jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/392772 )

Change subject: Use padding instead of margin to close a gap
......................................................................


Use padding instead of margin to close a gap

Clicking between the radio button and the logo should count as
a click on the thing.

Change-Id: I871fcbdd3a67e44be5a0ab2cec14e2d2ddbabb4c
---
M gateway_forms/mustache/forms.css
M gateway_forms/mustache/payment_method.html.mustache
2 files changed, 4 insertions(+), 8 deletions(-)

Approvals:
  XenoRyet: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/gateway_forms/mustache/forms.css b/gateway_forms/mustache/forms.css
index c74fd0f..cf0dc1e 100644
--- a/gateway_forms/mustache/forms.css
+++ b/gateway_forms/mustache/forms.css
@@ -274,14 +274,12 @@
 
 #cards.three-per-line li {
     width: 32.63%;
-    padding: 1em;
 }
 #cards.three-per-line > li:nth-child(3),
 #cards.three-per-line > li:nth-child(6) { margin-right: 0; }
 
 #cards.four-per-line li {
     width: 24.29%;
-    padding: 1em 0.5em;
 }
 #cards.four-per-line > li:nth-child(4),
 #cards.four-per-line > li:nth-child(8) { margin-right: 0; }
@@ -300,16 +298,16 @@
 #cards li input {
     display: block;
     width: 100%;
-    margin: 0 0 .7em;
+    margin: 0;
     box-shadow: none;
 }
 #cards li label {
     cursor: pointer;
-    margin-top: .5em;
+    padding: 1em;
     display: block;
 }
 #cards li label img {
-    margin: 0 auto;
+    margin: 0.7em auto 0;
     height: auto;
 }
 #cards li.has_sub_text label {
@@ -484,8 +482,6 @@
         border: 1px solid #ccc;
     }
     #cards li input.cardradio {
-        padding: 10px;
-        margin: 5px 0;
         display: block;
         width: auto;
         font-size: 13px;
diff --git a/gateway_forms/mustache/payment_method.html.mustache 
b/gateway_forms/mustache/payment_method.html.mustache
index 27e688a..b39a439 100644
--- a/gateway_forms/mustache/payment_method.html.mustache
+++ b/gateway_forms/mustache/payment_method.html.mustache
@@ -6,8 +6,8 @@
                                                                                
<ul class="options-h enabled {{ button_class }}" id="cards">
 {{# submethods }} {{! TODO: give every submethod a label_key, remove 
conditionals }}
                                                 <li title="{{# label_key }}{{ 
l10n . }}{{/ label_key }}{{^ label_key }}{{ label }}{{/ label_key }}" {{# 
sub_text_key }}class="has_sub_text" {{/ sub_text_key }}>
-                                                <input id="submethod-{{ key 
}}" name="payment_submethod" type="radio" value="{{ key }}" class="cardradio" />
                                                 <label for="submethod-{{ key 
}}">
+                                                    <input id="submethod-{{ 
key }}" name="payment_submethod" type="radio" value="{{ key }}" 
class="cardradio" />
                                                     <img 
class="submethod-logo" alt="{{ label }}" src="{{ logo }}" {{{ srcset }}}/>
 {{# sub_text_key }}
                                                     <p 
class="card_sub_text">{{ l10n . }}</p>

-- 
To view, visit https://gerrit.wikimedia.org/r/392772
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I871fcbdd3a67e44be5a0ab2cec14e2d2ddbabb4c
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/DonationInterface
Gerrit-Branch: master
Gerrit-Owner: Ejegg <ej...@ejegg.com>
Gerrit-Reviewer: AndyRussG <andrew.green...@gmail.com>
Gerrit-Reviewer: Mepps <me...@wikimedia.org>
Gerrit-Reviewer: XenoRyet <dkozlow...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to