Small html/layout issue

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Small html/layout issue

Jacopo Cappellato
Adrian, all,

there is a small html issue in the new layout that is causing a
validation warning message.
This is the code used to render a screenlet title bar:

<div class="screenlet-title-bar">
   <ul>
     <h3>${uiLabelMap.WebtoolsMainPage}</h3>
     <li class="disabled">${delegator.getDelegatorName()}</li>
   </ul>
   <br class="clear"/>
</div>

The warning message is caused by the <h3/> element because it could not
stay inside the <ul/> element; this is of course a minor issue but it
would be nice to try to generate 100% correct html pages.

Jacopo
Reply | Threaded
Open this post in threaded view
|

Re: Small html/layout issue

Adrian Crum
Jacopo,

Thank you for pointing that out. I'm aware it's not 100% valid. I did it that
way because I was trying to come up with a simple structure for everyone to use.

The logical alternative would be something like:

<div class="screenlet-title-bar">
   <ul>
     <li class="head3">${uiLabelMap.WebtoolsMainPage}</li>
     <li class="disabled">${delegator.getDelegatorName()}</li>
   </ul>
   <br class="clear"/>
</div>

The main style sheet and menu widget could be changed easily to support that
element compound.

-Adrian

Jacopo Cappellato wrote:

> Adrian, all,
>
> there is a small html issue in the new layout that is causing a
> validation warning message.
> This is the code used to render a screenlet title bar:
>
> <div class="screenlet-title-bar">
>   <ul>
>     <h3>${uiLabelMap.WebtoolsMainPage}</h3>
>     <li class="disabled">${delegator.getDelegatorName()}</li>
>   </ul>
>   <br class="clear"/>
> </div>
>
> The warning message is caused by the <h3/> element because it could not
> stay inside the <ul/> element; this is of course a minor issue but it
> would be nice to try to generate 100% correct html pages.
>
> Jacopo
>