[richfaces-issues] [JBoss JIRA] (RF-13346) orderingList - maxListHeight & minListHeight need to have set units along with number to work

[Michal Petrov (JIRA)]

    [ https://issues.jboss.org/browse/RF-13346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12923614#comment-12923614 ] 

Michal Petrov edited comment on RF-13346 at 11/14/13 9:08 AM:
--------------------------------------------------------------

The issue is with jQuery, and it's strange that it's not being handled. The problem is here:
{code}
_setHeightMin: function(height) {
    this.selectList.find('.scroll-box').css('min-height', height);
}
{code}

it doesn't work when height is just a number in string form, i.e.
* {{200}} - works
* {{"200px"}} - works
* {{"200"}} - does not work; yet it works for {{height}}

I'm guessing in RichWidgets the value is autocast to number, but in CDK string will remain a string. This will be an issue for every component. I will look for a workaround but I think the easiest way is to keep a list of options for each component and and parse them in the Bridge.
                
      was (Author: michpetrov):
    The issue is with jQuery, and it's strange that it's not being handled. The problem is here:
{code}
_setHeightMin: function(height) {
    this.selectList.find('.scroll-box').css('min-height', height);
}
{code}

it doesn't work when height is just a number in string form, i.e.
* {{200}} - works
* {{"200px"}} - works
* {{"200"}} - does not work

I'm guessing in RichWidgets the value is autocast to number, but in CDK string will remain a string. This will be an issue for every component. I will look for a workaround but I think the easiest way is to keep a list of options for each component and and parse them in the Bridge.
                  
> orderingList - maxListHeight & minListHeight need to have set units along with number to work
> ---------------------------------------------------------------------------------------------
>
>                 Key: RF-13346
>                 URL: https://issues.jboss.org/browse/RF-13346
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: component-selects
>    Affects Versions: 5.0.0.Alpha2
>            Reporter: Jiří Štefek
>            Assignee: Michal Petrov
>
> {{pickList}} attributes:
> * {{minListHeight}} and {{maxListHeight}}
> need to have set units along with the number value (e.g. 200px) in order to work correctly.
> In RF4 and in RichWidgets, these attributes can be left without units.

--
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