• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 3.1
• Affects Version/s: None
• Component/s: None
• Labels:

  None

1. 

   user-widget-bug.png

   34 kB

   11/Oct/10 17:19

[CMS-2236] The user widget layout is buggy

Collapse comment: Nicolas Gavalda (Inactive) added a comment - 12/Oct/10 16:12

Nicolas Gavalda (Inactive) added a comment - 12/Oct/10 16:12

It's definitely buggy, even defining a width when creating the object instance.
It seems the bug comes from this block :

org.ametys.cms.widgets.UserSelectorWidget.prototype.onRender = function(ct, position)
{
	org.ametys.cms.widgets.UserSelectorWidget.superclass.onRender.call(this, ct, position);
        [...]
	this.displayField = new org.ametys.HtmlContainer ({
		id: this.name + '-display',
		html: '<span class="empty">Cliquez ci-après pour sélectionnez un utilisateur</span>',
		cls: 'x-form-widget-display-field',
		hidden: false,
    	renderTo: this.wrap,
    	width: (this.getWidth() - 17)
    });

The "this.getWidth()" call does not return the wanted value, whereas the config should be already taken into account when getting there.

Moreover, the label "Cliquez ci-après pour sélectionnez un utilisateur" should be internationalized and comports a misspelling (should read "pour sélectionner").

Expand comment: Nicolas Gavalda (Inactive) added a comment - 12/Oct/10 16:12

Nicolas Gavalda (Inactive) added a comment - 12/Oct/10 16:12 It's definitely buggy, even defining a width when creating the object instance. It seems the bug comes from this block : org.ametys.cms.widgets.UserSelectorWidget. prototype .onRender = function (ct, position) { org.ametys.cms.widgets.UserSelectorWidget.superclass.onRender.call( this , ct, position); [...] this .displayField = new org.ametys.HtmlContainer ({ id: this .name + '-display' , html: '<span class= "empty" >Cliquez ci-après pour sélectionnez un utilisateur</span>' , cls: 'x-form-widget-display-field' , hidden: false , renderTo: this .wrap, width: ( this .getWidth() - 17) }); The "this.getWidth()" call does not return the wanted value, whereas the config should be already taken into account when getting there. Moreover, the label "Cliquez ci-après pour sélectionnez un utilisateur" should be internationalized and comports a misspelling (should read "pour sélectionner").

Collapse comment: Raphaël Franchet added a comment - 12/Oct/10 12:18

Raphaël Franchet added a comment - 12/Oct/10 12:18

You have to specify a width

Expand comment: Raphaël Franchet added a comment - 12/Oct/10 12:18

Raphaël Franchet added a comment - 12/Oct/10 12:18 You have to specify a width