Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-30 Thread Jacques Le Roux
Adrian, Inline From: Adrian Crum adrian.c...@sandglass-software.com Using the same logic, the page title should say Layout Demo (page-title) because it is rendered in the same way as h1. The whole point of the screen is to look at the markup - to understand how a screen's markup is composed

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-30 Thread Adrian Crum
Thanks Jacques! That is a good compromise. -Adrian On 1/30/2011 2:51 AM, Jacques Le Roux wrote: Adrian, Inline From: Adrian Crum adrian.c...@sandglass-software.com Using the same logic, the page title should say Layout Demo (page-title) because it is rendered in the same way as h1. The

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-29 Thread Adrian Crum
Bruno, At first glance, it might seem logical to have text describing each of the styles or elements being displayed, but it really isn't necessary - since the styles/elements can be seen when viewing the markup. -Adrian On 1/29/2011 6:22 AM, bus...@apache.org wrote: Author: buscob Date:

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-29 Thread Bruno Busco
Adrian, I thougth this was usefull above all when different styles are rendered in the same way (i.e. h4, h5, h6). In this case, having different texts, helps understanding very easily even if not looking at the markup. -Bruno 2011/1/29 Adrian Crum adrian.c...@sandglass-software.com Bruno,

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-29 Thread Adrian Crum
Using the same logic, the page title should say Layout Demo (page-title) because it is rendered in the same way as h1. The whole point of the screen is to look at the markup - to understand how a screen's markup is composed and how various styles are applied. In other words, if you want to

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-29 Thread Bruno Busco
Yes, you are right, the page title should be changed as well. I did not change it yet because I was using the Tomahawk theme and, in this theme, the page title is not visible because it is already present in the breadcrumb. The logic should be this: - for all styles that are rendered as simple

Re: svn commit: r1065018 - in /ofbiz/trunk/framework/webtools: config/WebtoolsUiLabels.xml widget/MiscScreens.xml

2011-01-29 Thread Adrian Crum
No, the page title should NOT be changed. You're missing the point. The extra text is redundant and unnecessary. Please do NOT clutter up a simple tool with a lot of unnecessary text. -Adrian On 1/29/2011 2:42 PM, Bruno Busco wrote: Yes, you are right, the page title should be changed as