Reduce the size of the "retractable" block settings

Bug #1417362 reported by Aaron Wells on 2015-02-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Wishlist
Aaron Wells

Bug Description

Every block currently has a pair of settings on the block config page to control whether the block is retractable, and if so, whether it should start out retracted. Because both of these settings have descriptions below them as well, they take up a fair bit of vertical space, even though they are seldom-used settings.

I'd like to explore some ways to make these settings more compact so that they don't expand the config screen so much.

Aaron Wells (u-aaronw) wrote :

A few strategies that I've considered:

1. Move these settings to the block layout section instead of having them in the block config. (i.e., next to the edit icon that opens the block config, we'd have some little icon to indicate that the block is collapsed/retractable)

2. Remove the "retractable" setting and only have the "automatically retracted" setting, the idea being that users are probably much more often interested in having a retracted block rather than a retractable block. But a scan of the data in a large client site found quite a few retractable but not automatically retracted blocks, so this probably isn't a good idea.

3. Consolidate the two checkboxes into one control with three states (off, retractable, automatically retracted).

I've pushed a patch to achieve strategy #3.

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

commit ea960f444d87fb762bdb6b11199b6aba6af0f7b3
Author: Aaron Wells <email address hidden>
Date: Tue Feb 3 16:17:11 2015 +1300

Consolidate the two block retraction checkboxes into one menu

Bug 1417362: Since it tends to be unwieldy to update block configdata
en masse, I've only merged the control, and I keep the retractonload
and rectrable settings separate in block_instance.configdata.

Change-Id: I56eaef5a161b0319b686ab222578538ab62e7873

Robert Lyon (robertl-9) on 2015-02-11
Changed in mahara:
status: In Progress → Fix Committed
tags: added: usermanualupdate
Robert Lyon (robertl-9) on 2015-04-17
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers