Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1490)

Issue 6941053: Fix GUI breakage on browser resizing.

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 4 months ago by frankban
Modified:
11 years, 4 months ago
Reviewers:
benjamin.saller, mp+139959
Visibility:
Public.

Description

Fix GUI breakage on browser resizing. Fixed how yui events are handled by the component framework. Also added two tests: - ensure that the module synthetic events binding works; - ensure that the custom events 'beforePageSizeRecalculation' and 'afterPageSizeRecalculation' are correctly fired by the environment view when the browser is resized. QA: resize the browser, open the charm panel, resize the browser again, close the charm panel, resize again, re-open the panel... https://code.launchpad.net/~frankban/juju-gui/bug-1090046-resize/+merge/139959 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix GUI breakage on browser resizing. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -4 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/assets/javascripts/d3-components.js View 1 chunk +6 lines, -3 lines 0 comments Download
M test/test_d3_components.js View 1 chunk +18 lines, -0 lines 0 comments Download
M test/test_environment_view.js View 2 chunks +17 lines, -1 line 0 comments Download

Messages

Total messages: 4
frankban
Please take a look.
11 years, 4 months ago (2012-12-14 16:40:36 UTC) #1
benjamin.saller
Thanks, This is a nice catch, a good fix and glad to see the additional ...
11 years, 4 months ago (2012-12-14 17:07:13 UTC) #2
gary.poster
Land as is. Thanks, Francesco! Great to have the user-facing fix and the general event ...
11 years, 4 months ago (2012-12-14 17:14:30 UTC) #3
frankban
11 years, 4 months ago (2012-12-14 17:20:20 UTC) #4
*** Submitted:

Fix GUI breakage on browser resizing.

Fixed how yui events are handled by the component framework.
Also added two tests:
- ensure that the module synthetic events binding works;
- ensure that the custom events 'beforePageSizeRecalculation' 
  and 'afterPageSizeRecalculation' are correctly fired by 
  the environment view when the browser is resized.

QA: resize the browser, open the charm panel, resize the 
    browser again, close the charm panel, resize again, 
    re-open the panel...

R=benjamin.saller, gary.poster
CC=
https://codereview.appspot.com/6941053
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b