[pmwiki-devel] Default Skin: Switch to CSS Table?

[H. Fox]

On Tue, Jun 3, 2014 at 4:10 PM, Petko Yotov <5ko at 5ko.fr> wrote:
> H. Fox writes:
>>
>> I've discovered a one-skin option to consider.
>>
>> Conditional Comments are only used by IE. We could use those to insert
>> the HTML table elements only for old versions of Internet Explorer.
>>
>> For example, this
>>
>>     <div id='wikileft'>
>>       <!--wiki:{$Group}.SideBar {$SiteGroup}.SideBar-->
>>     </div>
>>
>> becomes something like this
>>
>>     <div id='wikileft'>
>>     <!--[if lt IE 8 ]><td valign='top'><![endif]-->
>>       <!--wiki:{$Group}.SideBar {$SiteGroup}.SideBar-->
>>     <!--[if lt IE 8 ]></td><![endif]-->
>>     </div>
>
>
> MSIE<8 will see this exact markup as invalid, ie. div tags in the space
> between the <table>, <tr> and <td> tags. It will probably display fine
> though, or it can definitely be made valid with some tweaking.

Some tweakage has been applied.

   http://www.pmwiki.org/wiki/Skins/DefaultSkinWithoutTable#files

(No doubt it still would need more tweaking to be optimized.)

> But will this still be in line of the goals of the original skin? IIRC
> simple and easy to study and modify by a novice administrator?

It's hard to say.  Either way it becomes more complicated, just in a
different way.

> Recalling from previous discussions with Pm and the community, probably the
> least bad solution would be to add the (previous simpler) tableless skin
> with a different name.

If so, remove the border='0' attribute from the logo image's tag.

I'm pretty sure <img>s have no border by default in any browser
compatible with the CSS table, so the stylesheet can be left as-is
with the border='0' attribute removed.

Hagan