XHTML validation errors in webtrees theme header

[lindon]

Fix validation errors noted at http://validator.w3.org/check.

[kiwi (kiwi3685-deactivatedaccount)]

What is the actual validation error here? Wrapping a simple text element like ' | ' in <li></li> tags seems like overkill to me. If it's correct then other themes, especially fab will need to change as well.

[fisharebest (fisharebest)]

The error is that a "<ul>" may only contain "<li></li>"

Here, it contains a mixture of "<li></li>" and " | ".

So, the patch (if I read it correctly), it to replace one menu (containing lots of submenus, separated by pipes) with several menus, (each containing only one submenu), separated by a pipe.

An alternative solution might be to do this with CSS and use something along the lines of:

#whatever li content:before " | "
#whatever li:first content:before "";

[Rob Peters (makitso)]

Colors and Clouds had this some releases ago. However, we moved it all to css like Greg says. However, :after and :before do not work in IE7 so some extra padding between li's is necessary.

[kiwi (kiwi3685-deactivatedaccount)]

I agree that CSS would better, but if so then I would drop the '|' completely and use a left border.

[fisharebest (fisharebest)]

Or simply let IE7 users get the display without the |
(or perhaps some other puncutation, such as • )

[kiwi (kiwi3685-deactivatedaccount)]

Fixed - svn 11914.

[lindon (ycm05)]

Yes, CSS is definitely a better solution.

The last two changes in the patch were different validation errors however - see screenshot attached. Simply a space missing between parameters.
Regards,
lindon

[kiwi (kiwi3685-deactivatedaccount)]

Oops = I didn't look that far :-(

Fixed in svn 11918.

[fisharebest (fisharebest)]

Fix released in webtrees 1.2.2