Block configuration dialog is inconsistent width

Bug #1285862 reported by Jono M
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Invalid
High
Son Nguyen

Bug Description

In Mahara 1.8 the block configuration dialog is fairly narrow and expands to fit its content (if there is a file browser, HTML editor, etc.) In latest master the dialog can be anything from very narrow (where it cuts off the edge of the file browser) to very wide (where it fills the screen and leaves massive amounts of white space everywhere). I can't reliably reproduce any of these widths.
This is probably an issue in setDialogPosition() in views.js, which was changed when the page editor was made keyboard-accessible.

I've attached a couple of screenshots showing this inconsistency (not the most extreme examples, but you get the idea ...)

Tags: usability
Revision history for this message
Jono M (jonom) wrote :
Revision history for this message
Jono M (jonom) wrote :
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

We've been having this issue for some time. I believe it was some Javascript not working out well.

Son Nguyen (ngson2000)
Changed in mahara:
assignee: nobody → Son Nguyen (ngson2000)
status: New → In Progress
Aaron Wells (u-aaronw)
Changed in mahara:
milestone: 1.9.0 → 1.10.0
Revision history for this message
Nigel Cunningham (nigelc-g) wrote :

Hi Son.

Are you still working on this? I'm looking at bugs to address while waiting on other tasks :)

Nigel

Revision history for this message
Nigel Cunningham (nigelc-g) wrote :

I'm unable to reproduce this with current HEAD on the Master branch.

Revision history for this message
Son Nguyen (ngson2000) wrote :

Currently, the width of the configuration dialogue is calculated based on its content width. As its content can be dynamically loaded e.g. the file picker/browser, the dialogue width should be changed to make enough room to display. That's why its width is not consistent.

Revision history for this message
Aaron Wells (u-aaronw) wrote :

Marking this incomplete because we haven't got a specific reproducible test case for the current HEAD.

Changed in mahara:
status: In Progress → Incomplete
milestone: 1.10.0 → none
Revision history for this message
Robert Lyon (robertl-9) wrote :

This patch, that has been merged, deals with configure dialog being more flexible depending on viewport size allowing the configure dialog to work better for mobile devices

https://reviews.mahara.org/#/c/3728/

So will mark this as resolved

Changed in mahara:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.