You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by ca...@apache.org on 2018/03/11 11:47:59 UTC

[royale-asjs] 16/17: fix paddings and min heigh in buttons, but this should change again to em instead of px to handle responsiveness

This is an automated email from the ASF dual-hosted git repository.

carlosrovira pushed a commit to branch jewel-ui
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git

commit 8b49d1539189a5f33b0fb6779d564f71824d0c9d
Author: Carlos Rovira <ca...@apache.org>
AuthorDate: Sat Mar 10 11:33:42 2018 +0100

    fix paddings and min heigh in buttons, but this should change again to em instead of px to handle responsiveness
---
 frameworks/themes/JewelTheme/src/main/resources/defaults.css          | 4 ++--
 frameworks/themes/JewelTheme/src/main/resources/royale-jewel-blue.css | 4 ++--
 frameworks/themes/JewelTheme/src/main/resources/royale-jewel-red.css  | 4 ++--
 frameworks/themes/JewelTheme/src/main/sass/_button.sass               | 4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/frameworks/themes/JewelTheme/src/main/resources/defaults.css b/frameworks/themes/JewelTheme/src/main/resources/defaults.css
index da90252..0bc4b18 100644
--- a/frameworks/themes/JewelTheme/src/main/resources/defaults.css
+++ b/frameworks/themes/JewelTheme/src/main/resources/defaults.css
@@ -35,9 +35,9 @@
   cursor: pointer;
   display: inline-block;
   margin: 0;
-  padding: 10px 0px;
+  padding: 10px 16px;
   min-width: 74px;
-  min-height: 1em;
+  min-height: 34px;
   /* Background: */
   border: none;
   border-radius: 3px;
diff --git a/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-blue.css b/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-blue.css
index f748283..27242c2 100644
--- a/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-blue.css
+++ b/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-blue.css
@@ -35,9 +35,9 @@
   cursor: pointer;
   display: inline-block;
   margin: 0;
-  padding: 10px 0px;
+  padding: 10px 16px;
   min-width: 74px;
-  min-height: 1em;
+  min-height: 34px;
   /* Background: */
   border: none;
   border-radius: 3px;
diff --git a/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-red.css b/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-red.css
index 3bda370..825f537 100644
--- a/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-red.css
+++ b/frameworks/themes/JewelTheme/src/main/resources/royale-jewel-red.css
@@ -35,9 +35,9 @@
   cursor: pointer;
   display: inline-block;
   margin: 0;
-  padding: 10px 0px;
+  padding: 10px 16px;
   min-width: 74px;
-  min-height: 1em;
+  min-height: 34px;
   /* Background: */
   border: none;
   border-radius: 3px;
diff --git a/frameworks/themes/JewelTheme/src/main/sass/_button.sass b/frameworks/themes/JewelTheme/src/main/sass/_button.sass
index 9d78f0f..ca39960 100644
--- a/frameworks/themes/JewelTheme/src/main/sass/_button.sass
+++ b/frameworks/themes/JewelTheme/src/main/sass/_button.sass
@@ -25,9 +25,9 @@
 	
 	display: inline-block
 	margin: 0 //1rem
-	padding: 10px 0px //.938em 1.875em
+	padding: 10px 16px //.938em 1.875em
 	min-width: $button-min-width
-	//min-height: 1em
+	min-height: 34px
 
 	/* Background: */
 	border: none // .094em solid 

-- 
To stop receiving notification emails like this one, please contact
carlosrovira@apache.org.