• 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

Nicolas Gavalda (Inactive) made changes - 22/Dec/10 18:42

Status

Original: Resolved [ 5 ]

New: Closed [ 6 ]

Raphaël Franchet made changes - 12/Oct/10 17:43

Resolution		New: Fixed [ 1 ]
Status	Original: Open [ 1 ]	New: Resolved [ 5 ]

Raphaël Franchet made changes - 12/Oct/10 17:43

Link

New: This issue Bloque MYPAGE-30 [ MYPAGE-30 ]

Nicolas Gavalda (Inactive) made changes - 12/Oct/10 16:12

Assignee

Original: Nicolas Gavalda [ ngavalda ]

New: Raphaël Franchet [ raphael ]

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").

Nicolas Gavalda (Inactive) made changes - 12/Oct/10 16:05

Fix Version/s		New: 3.1 [ 10251 ]
Fix Version/s	Original: 1.1 [ 10279 ]
Key	Original: MYPAGE-29	New: CMS-2236
Project	Original: My Page [ 10071 ]	New: CMS [ 10021 ]
Workflow	Original: jira [ 15965 ]	New: Ametys [ 15966 ]

Raphaël Franchet made changes - 12/Oct/10 12:18

Assignee

New: Nicolas Gavalda [ ngavalda ]

Raphaël Franchet made changes - 12/Oct/10 12:18

Fix Version/s

New: 1.1 [ 10279 ]

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

Raphaël Franchet made changes - 12/Oct/10 12:18

Component/s	Original: UI [ 10061 ]
Fix Version/s	Original: 3.1 [ 10251 ]
Key	Original: CMS-2235	New: MYPAGE-29
Project	Original: CMS [ 10021 ]	New: My Page [ 10071 ]
Workflow	Original: Ametys [ 15963 ]	New: jira [ 15965 ]

Load more older events