[TAP5-2122] visible() on empty <div> is false; prevents valiation of Palette component

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 5.4
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Labels: jquery validation
• Assignee: Howard Lewis Ship
• Reporter: Howard Lewis Ship
• Created: 30/May/13 00:25
• Updated: 05/Jun/13 09:58
• Resolved: 31/May/13 23:07

Description

Noticed that required validation for Palette was not working; a little debugging shows that the validation disabled because the <select> element is not deepVisible().

The enclosing div.t-palette is not jQuery.is(":visible"); that's because it has no text content (I believe), just a hidden field and nested divs.

The correct solution is that visible() should be based on the CSS "display" property not being "hidden".