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 2020/08/09 13:29:26 UTC

[royale-asjs] branch develop updated: jewel-sectioncontent: fix an issue with sizing in components inside SectionContent containers. Since SC default state is hidden, that made components inside report 0 for width and height instead getting the right size, This should hopefully solve other layout issues

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

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


The following commit(s) were added to refs/heads/develop by this push:
     new 2af3137  jewel-sectioncontent: fix an issue with sizing in components inside SectionContent containers. Since SC default state is hidden, that made components inside report 0 for width and height instead getting the right size, This should hopefully solve other layout issues
2af3137 is described below

commit 2af3137baccfcc2df5ea500edef4b8eced3a320a
Author: Carlos Rovira <ca...@apache.org>
AuthorDate: Sun Aug 9 15:29:12 2020 +0200

    jewel-sectioncontent: fix an issue with sizing in components inside SectionContent containers. Since SC default state is hidden, that made components inside report 0 for width and height instead getting the right size, This should hopefully solve other layout issues
---
 frameworks/projects/Jewel/src/main/resources/defaults.css            | 5 +++--
 .../projects/Jewel/src/main/sass/components/_sectioncontent.sass     | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/frameworks/projects/Jewel/src/main/resources/defaults.css b/frameworks/projects/Jewel/src/main/resources/defaults.css
index c52645f..c0592b0 100644
--- a/frameworks/projects/Jewel/src/main/resources/defaults.css
+++ b/frameworks/projects/Jewel/src/main/resources/defaults.css
@@ -3504,7 +3504,8 @@ j|PopUp {
   }
 }
 .jewel.section {
-  display: none;
+  display: flex;
+  visibility: hidden;
   flex-flow: column nowrap;
   height: 100%;
 }
@@ -3512,7 +3513,7 @@ j|PopUp {
   flex: 0 0 auto;
 }
 .jewel.section.is-selected {
-  display: flex;
+  visibility: visible;
 }
 
 j|SectionContent {
diff --git a/frameworks/projects/Jewel/src/main/sass/components/_sectioncontent.sass b/frameworks/projects/Jewel/src/main/sass/components/_sectioncontent.sass
index 8fe2f99..418382b 100644
--- a/frameworks/projects/Jewel/src/main/sass/components/_sectioncontent.sass
+++ b/frameworks/projects/Jewel/src/main/sass/components/_sectioncontent.sass
@@ -22,7 +22,8 @@
 // SectionContent variables
 
 .jewel.section
-    display: none
+    display: flex
+    visibility: hidden
     flex-flow: column nowrap
     height: 100%
 
@@ -30,7 +31,7 @@
         flex: 0 0 auto
     
     &.is-selected
-        display: flex
+        visibility: visible
 
 j|SectionContent
     IBeadLayout: ClassReference("org.apache.royale.jewel.beads.layouts.NullLayout")