Replace access setting checkboxes with radio buttons

Bug #638651 reported by Tom Hoffman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SchoolTool
Fix Released
High
Justas Sadzevičius
SchoolTool Gradebook
Fix Released
High
Justas Sadzevičius

Bug Description

Instead of checkboxes, we want two way radio buttons with each state defined. The default is marked by [X] Also define "school administration" as it is in the access rights page

Also, having gone through all these, I'm not sure what the most intuitive order of the choices is. Default selection first? More restrictive first? Less?

Everyone can view the application calendar. ->

Actually, this is stupid, remove this choice so it is always viewable by everybody. Although I suppose that might cause some small evolution headaches... either way, just remove the option.

Everyone can view section information and calendars ->

[ ] Basic information about sections, including rosters, is visible to the general public.
[X] Basic information about sections is only visible to instructors and students in that section, and school administration.

Everyone can view the list of persons ->

[ ] The list of people with accounts in the system is visible to the general public.
[X] The list of people with accounts in the system is visible only to school administration.

Everyone can view info of persons -> Realistically, we should just remove this option.

Everyone can set their own preferences ->

[X] Users can make their own calendars public.
[ ] Users cannot make their own calendars public.

Persons can change their own passwords ->

[X] Users can change their own passwords
[ ] Only school administration can change passwords

Everyone can view group lists ->
[ ] The list of groups in the system is visible to the general public.
[X] Only school administration can view the list of groups.

Everyone can view group information ->
[ ] Information about each group is visible to the general public.
[X] Information about each group is visible to school administration, and members and leaders of the group.

Everyone can view group calendars ->
[ ] Each group calendar is visible to the general public.
[X] Each group calendar is visible to school administration, and members and leaders of the group.

Group members can edit group calendars ->
[ ] All members of a group and school administration can edit its calendar.
[X] The leaders of a group and school administration can edit its calendar.

Everyone can view resource lists ->
[ ] The list of resources in the system is visible to the general public.
[X] Only school administration can view the list of resources.

Everyone can view resource information.
[ ] Information about each resource is visible to the general public.
[X] Information about each resource is visible to school administration and leaders of the resource.

Everyone can view resource calendars.
[ ] Resource calendars are visible to the general public
[X] Resource calendars are visible to school administration, teachers, and leaders of the resource.

Administration can grade students
[X] Only the instructor of a section can edit its gradebook.
[ ] The instructor of a section and school administration can edit a section's gradebook.

Revision history for this message
Justas Sadzevičius (justas.sadzevicius) wrote :

This is a very nice intermediate step! The only concern is that we're introducing a lot of untranslated strings just before release.

Another enhancement to access rights I'd like to see in a reasonably near release - basically change list bullets in the access rights view to checkboxes, for example:

Groups /
Change calendar
[V] School administration [1]
[ ] Group members
[V] Assigned leaders.

Hmm, also there's some concern to keep the strings up-to-date. For example, the option of "Resource calendars are visible to school administration, teachers, and leaders of the resource". We either put the string as it is and manually check those every time permissions are changed in ZCML, or auto-generate it as access rights list does. Auto-generation is trickier to do than you'd think.

Revision history for this message
Tom Hoffman (tom-hoffman) wrote : Re: [Bug 638651] Re: Update Access Control wording

You have a point about the strings, but let's just go ahead and do it.
 Also, we'll just have to manually maintain them. It isn't something we
tweak constantly or lightly.

--Tom

2010/9/15 Justas Sadzevičius <email address hidden>

> This is a very nice intermediate step! The only concern is that we're
> introducing a lot of untranslated strings just before release.
>
> Another enhancement to access rights I'd like to see in a reasonably
> near release - basically change list bullets in the access rights view
> to checkboxes, for example:
>
> Groups /
> Change calendar
> [V] School administration [1]
> [ ] Group members
> [V] Assigned leaders.
>
> Hmm, also there's some concern to keep the strings up-to-date. For
> example, the option of "Resource calendars are visible to school
> administration, teachers, and leaders of the resource". We either put
> the string as it is and manually check those every time permissions are
> changed in ZCML, or auto-generate it as access rights list does. Auto-
> generation is trickier to do than you'd think.
>
> --
> Update Access Control wording
> https://bugs.launchpad.net/bugs/638651
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Justas Sadzevičius (justas.sadzevicius) wrote : Re: Update Access Control wording

"Administration can grade students" comes from the gradebook

affects: schooltool → schooltool.gradebook
Changed in schooltool.gradebook:
milestone: 1.5.2 → none
affects: schooltool.gradebook → schooltool
Changed in schooltool:
milestone: none → 1.5.2
Changed in schooltool.gradebook:
importance: Undecided → High
assignee: nobody → Justas Sadzevičius (justas-pov)
milestone: none → 0.8.1
Changed in schooltool:
status: New → Fix Committed
Changed in schooltool.gradebook:
status: New → Fix Committed
summary: - Update Access Control wording
+ Replace access setting checkboxes with radio buttons
Changed in schooltool:
status: Fix Committed → Fix Released
Changed in schooltool.gradebook:
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.