when editing a file download artefact and the licence acceptance is on, the entire screen is used

[Hugh Davenport]

using chrome 21.0.1180.89 Debian wheezy/sid (154005)

It uses the entire screen, whereas I would think it should use a text wrap and use the normal size (without the agreement).

[Hugh Davenport (hugh-davenport)]

Designer has said that is to do with js using a calculated px for width, it should be changed to have %

[Kristina Hoeppner (kris-hoeppner)]

It is still an issue also in Mahara 1.10 dev.

Steps to replicate:

1. Turn on the upload agreement in Admin -> Plugins -> File configuration.
2. Add any file block. I added the image block and got the attached result in "mahara-1-10_image_license_acceptance_on.png".
3. Turn off the upload agreement.
4. Add any file block. I added the image block and got the attached result in "mahara-1-10_image_license_acceptance_off.png", which is the pop-up in the regular width.

[Mahara Bot (dev-mahara)]

Patch for "master" branch: https://reviews.mahara.org/3728

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/3728
Committed: http://gitorious.org/mahara/mahara/commit/141ee71b4a56c14023a7b3f2b7932934895bce0c
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 141ee71b4a56c14023a7b3f2b7932934895bce0c
Author: Nigel Cunningham <email address hidden>
Date: Mon Dec 29 09:30:10 2014 +1100

Bug1051844: Fix block configure dialog width.

The configure dialog for blocks is currently set to a fixed width via
Javascript, but there is no handler for browser resizing and it results in the
dialogue being very wide for the sake of just one line when a license agreement
is enabled.

Fix this issue by making all dimensions percentages, and using a combination of
max-width: 90% and width: max-content. In addition, make the dialog fixed
position with its own scrollbars as needed (the body's scrollbar is disabled
when the dialog is shown).

Contextual help's width code and CSS is also adjusted so that it wraps within
dialog boxes rather than pushing the width out.

Change-Id: If2a40739f1510cedcf9a6133ccfb8f0a18257356

[Robert Lyon (robertl-9)]

The refactoring of the configure form popup is now controlled via css and is far more flexible - allowing for resizing the screen and the configure block adjusting accordingly.