RE: TabbedPanel whitspace markup

2012-09-18 Thread Lemke, Michael SZ/HZA-ZSW
On Tue, Sep 18, 2012 2:10 PM
Martin Grigorov [mailto:mgrigo...@apache.org] wrote:
>To: users@wicket.apache.org
>Subject: Re: TabbedPanel whitspace markup
>
>Create a ticket and attach the screenshots.
>I'm not much in CSS and I cannot imagine how this whitespace breaks
>the layout. But I don't see a problem to remove them too.

Thanks Martin.  Created
https://issues.apache.org/jira/browse/WICKET-4768

>
>On Tue, Sep 18, 2012 at 3:07 PM, Lemke, Michael  SZ/HZA-ZSW
> wrote:
>> On Tue, Sep 18, 2012 1:51 PM
>> Martin Grigorov wrote:
>>>On Tue, Sep 18, 2012 at 2:29 PM, Lemke, Michael  SZ/HZA-ZSW wrote:
>>>> I've encountered a layout problem with TabbedPanels.  In the
>>>> generated markup there is whitespace between the LI items and
>>>> the anchor that switches tabs.  This causes a layout problem for me
>>>> with this style sheet (simplified to show the issue, not tested):
>>>>
>>>> .selected a {
>>>> background: url("../i/bg_tab_lw_content_active.gif") repeat-x scroll 
>>>> center top #E8EEF1;
>>>> border-right: 1px solid #CDD3D6;
>>>> }
>>>>
>>>> a {
>>>> background: none repeat scroll 0 0 transparent;
>>>> border-right: 1px solid #FF;
>>>> }
>>>>
>>>> li {
>>>> display: inline;
>>>> margin: 0;
>>>> list-style: none outside none
>>>> }
>>>>
>>>> The idea is to have a white border between the tabs which
>>>> disappears for the selected tab.  Kind of hard to explain
>>>> without screenshots.  Currently with Wicket there is always
>>>> a small margin left or right of the white border of the A
>>>> element.  It is caused by whitespace in the markup.
>>>>
>>>> li { float:left; } would remove the undesired whitespace
>>>> but would break the rest of our layout.
>>>>
>>>>
>>>> Anyway, the root cause is the markup in TabbedPanel.html of
>>>> Wicket.
>>>>
>>>> Current (1.5.8 and 6.0.0):
>>>>
>>>> http://wicket.apache.org";>
>>>> 
>>>> 
>>>> 
>>>> [[tab 
>>>> title]]
>>>> 
>>>> 
>>>> 
>>>> 
>>>> 
>>>>
>>>>
>>>> Should be changed in e.g.
>>>>
>>>> http://wicket.apache.org";>
>>>> 
>>>> 
>>>> >>> wicket:id="title">[[tab title]]
>>>> 
>>>> 
>>>> 
>>>> 
>>>>
>>>>
>>>> Is there a Wicket way to work around this?
>>>
>>>You can create MyTabbedPanel that extends TabbedPanel and provides
>>>MyTabbedPanel.html.
>>>The only code in .java that is needed to add is the constructor(s) you use.
>>>
>>
>> Thanks, I just did this.  Works fine.  But is there a chance
>> it gets fixed in Wicket?  Or isn't this considered a problem?
>>
>> Michael
>
>
>
>-- 
>Martin Grigorov
>jWeekend
>Training, Consulting, Development
>http://jWeekend.com
>
>-
>To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
>For additional commands, e-mail: users-h...@wicket.apache.org
>
>

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org



Re: TabbedPanel whitspace markup

2012-09-18 Thread Martin Grigorov
Create a ticket and attach the screenshots.
I'm not much in CSS and I cannot imagine how this whitespace breaks
the layout. But I don't see a problem to remove them too.

On Tue, Sep 18, 2012 at 3:07 PM, Lemke, Michael  SZ/HZA-ZSW
 wrote:
> On Tue, Sep 18, 2012 1:51 PM
> Martin Grigorov wrote:
>>On Tue, Sep 18, 2012 at 2:29 PM, Lemke, Michael  SZ/HZA-ZSW wrote:
>>> I've encountered a layout problem with TabbedPanels.  In the
>>> generated markup there is whitespace between the LI items and
>>> the anchor that switches tabs.  This causes a layout problem for me
>>> with this style sheet (simplified to show the issue, not tested):
>>>
>>> .selected a {
>>> background: url("../i/bg_tab_lw_content_active.gif") repeat-x scroll 
>>> center top #E8EEF1;
>>> border-right: 1px solid #CDD3D6;
>>> }
>>>
>>> a {
>>> background: none repeat scroll 0 0 transparent;
>>> border-right: 1px solid #FF;
>>> }
>>>
>>> li {
>>> display: inline;
>>> margin: 0;
>>> list-style: none outside none
>>> }
>>>
>>> The idea is to have a white border between the tabs which
>>> disappears for the selected tab.  Kind of hard to explain
>>> without screenshots.  Currently with Wicket there is always
>>> a small margin left or right of the white border of the A
>>> element.  It is caused by whitespace in the markup.
>>>
>>> li { float:left; } would remove the undesired whitespace
>>> but would break the rest of our layout.
>>>
>>>
>>> Anyway, the root cause is the markup in TabbedPanel.html of
>>> Wicket.
>>>
>>> Current (1.5.8 and 6.0.0):
>>>
>>> http://wicket.apache.org";>
>>> 
>>> 
>>> 
>>> [[tab 
>>> title]]
>>> 
>>> 
>>> 
>>> 
>>> 
>>>
>>>
>>> Should be changed in e.g.
>>>
>>> http://wicket.apache.org";>
>>> 
>>> 
>>> >> wicket:id="title">[[tab title]]
>>> 
>>> 
>>> 
>>> 
>>>
>>>
>>> Is there a Wicket way to work around this?
>>
>>You can create MyTabbedPanel that extends TabbedPanel and provides
>>MyTabbedPanel.html.
>>The only code in .java that is needed to add is the constructor(s) you use.
>>
>
> Thanks, I just did this.  Works fine.  But is there a chance
> it gets fixed in Wicket?  Or isn't this considered a problem?
>
> Michael



-- 
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org



RE: TabbedPanel whitspace markup

2012-09-18 Thread Lemke, Michael SZ/HZA-ZSW
On Tue, Sep 18, 2012 1:51 PM
Martin Grigorov wrote:
>On Tue, Sep 18, 2012 at 2:29 PM, Lemke, Michael  SZ/HZA-ZSW wrote:
>> I've encountered a layout problem with TabbedPanels.  In the
>> generated markup there is whitespace between the LI items and
>> the anchor that switches tabs.  This causes a layout problem for me
>> with this style sheet (simplified to show the issue, not tested):
>>
>> .selected a {
>> background: url("../i/bg_tab_lw_content_active.gif") repeat-x scroll 
>> center top #E8EEF1;
>> border-right: 1px solid #CDD3D6;
>> }
>>
>> a {
>> background: none repeat scroll 0 0 transparent;
>> border-right: 1px solid #FF;
>> }
>>
>> li {
>> display: inline;
>> margin: 0;
>> list-style: none outside none
>> }
>>
>> The idea is to have a white border between the tabs which
>> disappears for the selected tab.  Kind of hard to explain
>> without screenshots.  Currently with Wicket there is always
>> a small margin left or right of the white border of the A
>> element.  It is caused by whitespace in the markup.
>>
>> li { float:left; } would remove the undesired whitespace
>> but would break the rest of our layout.
>>
>>
>> Anyway, the root cause is the markup in TabbedPanel.html of
>> Wicket.
>>
>> Current (1.5.8 and 6.0.0):
>>
>> http://wicket.apache.org";>
>> 
>> 
>> 
>> [[tab 
>> title]]
>> 
>> 
>> 
>> 
>> 
>>
>>
>> Should be changed in e.g.
>>
>> http://wicket.apache.org";>
>> 
>> 
>> > wicket:id="title">[[tab title]]
>> 
>> 
>> 
>> 
>>
>>
>> Is there a Wicket way to work around this?
>
>You can create MyTabbedPanel that extends TabbedPanel and provides
>MyTabbedPanel.html.
>The only code in .java that is needed to add is the constructor(s) you use.
>

Thanks, I just did this.  Works fine.  But is there a chance
it gets fixed in Wicket?  Or isn't this considered a problem?

Michael


Re: TabbedPanel whitspace markup

2012-09-18 Thread Martin Grigorov
Hi,

On Tue, Sep 18, 2012 at 2:29 PM, Lemke, Michael  SZ/HZA-ZSW
 wrote:
> I've encountered a layout problem with TabbedPanels.  In the
> generated markup there is whitespace between the LI items and
> the anchor that switches tabs.  This causes a layout problem for me
> with this style sheet (simplified to show the issue, not tested):
>
> .selected a {
> background: url("../i/bg_tab_lw_content_active.gif") repeat-x scroll 
> center top #E8EEF1;
> border-right: 1px solid #CDD3D6;
> }
>
> a {
> background: none repeat scroll 0 0 transparent;
> border-right: 1px solid #FF;
> }
>
> li {
> display: inline;
> margin: 0;
> list-style: none outside none
> }
>
> The idea is to have a white border between the tabs which
> disappears for the selected tab.  Kind of hard to explain
> without screenshots.  Currently with Wicket there is always
> a small margin left or right of the white border of the A
> element.  It is caused by whitespace in the markup.
>
> li { float:left; } would remove the undesired whitespace
> but would break the rest of our layout.
>
>
> Anyway, the root cause is the markup in TabbedPanel.html of
> Wicket.
>
> Current (1.5.8 and 6.0.0):
>
> http://wicket.apache.org";>
> 
> 
> 
> [[tab 
> title]]
> 
> 
> 
> 
> 
>
>
> Should be changed in e.g.
>
> http://wicket.apache.org";>
> 
> 
>  wicket:id="title">[[tab title]]
> 
> 
> 
> 
>
>
> Is there a Wicket way to work around this?

You can create MyTabbedPanel that extends TabbedPanel and provides
MyTabbedPanel.html.
The only code in .java that is needed to add is the constructor(s) you use.

>
> Thanks for any insight,
> Michael
>
>
> -
> To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
> For additional commands, e-mail: users-h...@wicket.apache.org
>



-- 
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org