adjust width in configure block

Bug #1381715 reported by Dirk Meyer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Unassigned
1.10
Fix Released
High
Aaron Wells
15.04
Fix Released
High
Unassigned

Bug Description

1.10rc1

Presently width=500px. If at 625px, editor icons display nicer. See attachments

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

Thanks Dirk!

Widening the block may have an impact on usability for portable devices. Another option would be to move the buttons into four rows instead of three.

Cheers,
Aaron

Changed in mahara:
milestone: none → 1.10.0
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Aaron Wells (u-aaronw) wrote :

I emailed our design expert Evonne to get her feedback on whether it'd better to widen the config popup or wrap the buttons into 4 narrow rows.

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

I heard back from Evonne. She said it would be fine to widen it to 625px.

She also suggested just putting all the buttons in one big row and letting them word-wrap into as many rows as necessary. I tried a form of this -- making the buttons you see when you hit the "advanced" toggle into one wrappable row. It looked good on block configs, but it caused the TinyMCE area to get *too* wide on other pages, which did not look good.

So I'm just going to go with the 625px widening.

Cheers,
Aaron

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "1.10_STABLE" branch: https://reviews.mahara.org/3838

Revision history for this message
Dirk Meyer (dirkca68) wrote :

Hi Aaron,
sorry I am getting late to this again.

The Add Journal textbox (/artefact/blog/new/index.php) works well for me. I am just realizing this now. Maybe its a better fix to apply to the add textbox issue I reported here?

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/3838
Committed: http://gitorious.org/mahara/mahara/commit/a6f6e6509ad85b175e4ce905895b2fa353c893a0
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.10_STABLE

commit a6f6e6509ad85b175e4ce905895b2fa353c893a0
Author: Aaron Wells <email address hidden>
Date: Mon Oct 20 12:48:46 2014 +1300

Widen block configs so TinyMCE won't wrap

Bug 1381715

Change-Id: I30fed711addac13be086eee08228fd4057204f8d

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

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

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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

commit 28b194d82a1b6ee3f16985b97435759536fa0047
Author: Aaron Wells <email address hidden>
Date: Mon Oct 20 12:48:46 2014 +1300

Widen block configs so TinyMCE won't wrap

Bug 1381715

Change-Id: I30fed711addac13be086eee08228fd4057204f8d

Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
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.