You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2013/06/01 01:48:19 UTC

[jira] [Commented] (TAP5-2122) visible() on empty
is false; prevents valiation of Palette component

    [ https://issues.apache.org/jira/browse/TAP5-2122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13671905#comment-13671905 ] 

Hudson commented on TAP5-2122:
------------------------------

Integrated in tapestry-trunk-freestyle #1070 (See [https://builds.apache.org/job/tapestry-trunk-freestyle/1070/])
    TAP5-2122: visible() on empty <div> is false (Revision 47c83bb8374bc12e1c915e5b62c34b8c57ba5a53)

     Result = SUCCESS
hlship : 
Files : 
* tapestry-core/src/main/coffeescript/org/apache/tapestry5/t5-core-dom-jquery.coffee

                
> visible() on empty <div> is false; prevents valiation of Palette component
> --------------------------------------------------------------------------
>
>                 Key: TAP5-2122
>                 URL: https://issues.apache.org/jira/browse/TAP5-2122
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.4
>            Reporter: Howard M. Lewis Ship
>            Assignee: Howard M. Lewis Ship
>            Priority: Blocker
>              Labels: jquery, validation
>             Fix For: 5.4
>
>
> 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".

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira