779434 - backport bug 437212 to bmo (refreshing a page does not update fields)

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: User Interface, defect)

Description

[:glob ✱]

the review on upstream bug 437212 appears stalled; i'd love to get this deployed on bmo sooner rather than later so as discussed i'll backport this now.

Comment 1

[:glob ✱]

Attachment: patch v1

Comment 2

[David Lawrence [:dkl]]

Comment on attachment 647885 [details] [diff] [review]
patch v1

Review of attachment 647885 [details] [diff] [review]:
-----------------------------------------------------------------

Tested and works as expected. r=dkl

::: js/show_bug.js
@@ +21,5 @@
> +// Force the browser to honour the selected option when a page is refreshed,
> +// but if the user hasn't explicitly selected a different option.
> +YAHOO.util.Event.onDOMReady(function() {
> +    var selects = document.getElementById('changeform').getElementsByTagName('select');
> +    for (var i = 0, l = selects.length; i < l; i++) {

nit: Why not just i < selects.length? I do not see where l is needed anywhere else. Easier to understand this way?

Comment 3

[:glob ✱]

(In reply to David Lawrence [:dkl] from comment #2)
> > +    var selects = document.getElementById('changeform').getElementsByTagName('select');
> > +    for (var i = 0, l = selects.length; i < l; i++) {
> 
> nit: Why not just i < selects.length? I do not see where l is needed
> anywhere else. Easier to understand this way?

as the length of selects doesn't change while iterating, i put it into the initialisation section of the for.  i've hit performance issues by doing for(i = 0; i < a.length; i++) in the past, especially when dealing with a dom list, as this requires the length to be recalculated with each pass of the loop.

Comment 4

[David Lawrence [:dkl]]

(In reply to Byron Jones ‹:glob› from comment #3)
> as the length of selects doesn't change while iterating, i put it into the
> initialisation section of the for.  i've hit performance issues by doing
> for(i = 0; i < a.length; i++) in the past, especially when dealing with a
> dom list, as this requires the length to be recalculated with each pass of
> the loop.

Ah thanks. Never considered the performance implication of using .length over and over. So with that I learn something new :)

dkl

Comment 5

[:glob ✱]

Committing to: bzr+ssh://[e-mail]/bmo/4.0/
added js/show_bug.js
modified template/en/default/bug/edit.html.tmpl
modified template/en/default/bug/field.html.tmpl
modified template/en/default/bug/show-header.html.tmpl
modified template/en/default/flag/list.html.tmpl
Committed revision 8264.

Committing to: bzr+ssh://[e-mail]/bmo/4.2/
added js/show_bug.js
modified template/en/default/bug/edit.html.tmpl
modified template/en/default/bug/field.html.tmpl
modified template/en/default/bug/show-header.html.tmpl
modified template/en/default/flag/list.html.tmpl
Committed revision 8281.