[2.2][FIXED] Combobox position failure (hideTrigger)

[Jacka]

A single Combobox will be displayed on the top of the page, if 'hideTrigger' is set on true.
With 'false' it works normal. Tested IE 6.0 and IE 7.
Firefox 3 works normal.

[brian.moeskau]

Try this:

Ext.form.TriggerField.override({
    afterRender : function(){
        Ext.form.TriggerField.superclass.afterRender.call(this);
        var y;
        if(Ext.isIE && !this.hideTrigger && this.el.getY() != (y = this.trigger.getY())){
            this.el.position();
            this.el.setY(y);
        }
    }
});

[brian.moeskau]

Fixed in SVN, thanks.

[Yak52]

The 'DateFiled' is have a such problem if the 'hideTrigger' is set on true.

[Condor]

The fix listed above is wrong. It should be:

Ext.form.TriggerField.override({
    afterRender : function(){
        Ext.form.TriggerField.superclass.afterRender.call(this);
        var y;
        if(Ext.isIE && !this.hideTrigger && this.el.getY() != (y = this.trigger.getY())){
            this.el.position();
            this.el.setY(y);
        }
    }
});

(SVN is correct)

[brian.moeskau]

Oops

Yes, it applies to all TriggerFields. I even had the superclass call correct...

[Jacka]

Thank you!

Kind regards,
Jacka

[Zecc]

Yes, thank you!

It's interesting how I found this thread exactly after I was preparing to drop the hideTrigger: true from my code...

Of course, now I need to discover why this is happening:

Edit: ah, I've found a quickfix in another thread.

[bobvandell]

Any one know the cause for this bug? (Thanks for the fix)

[mmusson]

Originally Posted by bobvandell

Any one know the cause for this bug? (Thanks for the fix)

Yes, it is straightforward. The trigger element is hidden with a call to Element setDisplayed(false). This has the effect of setting the CSS style display:none. This removes the element from the HTML layout.

The IE specific code above is getting the y position of the trigger element. The y position of the trigger element is 0 when it has the CSS style display:none causing the text part of the trigger field to be incorrectly positioned. Adding a check for hideTrigger keeps the code from running in this case and causing problems.