Bug #3465
closedWhen popups are too tall or too wide, they are not displayed correctly
Description
Popups in Rudder grow very wide and tall with workflows activated ( Rules impacted + workflow + message = too much to be displayed)
We need to change the way we create those popups to allow them to be correctly displayed (adatped to height and width, then resized and centered to the page)
This bug only concerns 2.6 because popups get big enough only since then.
Updated by Vincent MEMBRÉ over 11 years ago
- Status changed from New to Pending technical review
Pull request here : https://github.com/Normation/rudder/pull/167
Updated by Vincent MEMBRÉ over 11 years ago
- Assignee changed from Vincent MEMBRÉ to François ARMAND
Updated by Vincent MEMBRÉ over 11 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset 8736b66d2df25ee16a98f64b1aca65e4f35eaea0.
Updated by Anonymous over 11 years ago
Applied in changeset b79e48f12daf87d12cbd9603fdec019bd9f72ce2.
Updated by Vincent MEMBRÉ over 11 years ago
Another pull request was made to make some other js correction https://github.com/Normation/rudder/pull/172
Updated by Nicolas CHARLES over 11 years ago
Hum, just to be sure, doesn't it break some layout for popup ? I thought we had some kind of "policy" for popup width.height, that might get broken with this implementation, isn't it ?
Updated by Vincent MEMBRÉ over 11 years ago
by setting autoresize to true in the modal call, the size is dinamcally computed when resize events.
Before we had awful problems with huge popups (group/directive modification with change request activated...) and they were not visible in the browser.
AutoResize make it all work for us and adapt it to height/width of the window (can display popup on very very small browser and still access buttons and the top right corner close button)
The key is to call the resize.simplemodal event every time we want to update our popup
Updated by Jonathan CLARKE over 11 years ago
- Status changed from Pending release to Released