Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-12120

Assign roles role overrides would be better as two separate tabs next to 'update'

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 1.9
    • Component/s: Roles / Access
    • Labels:
      None

      Description

      When updating an activity, it would be better to have three tabs at the top rather than have to click twice for role overrides. Especially as its not clear where overrides is without this or that it is available.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            dougiamas Martin Dougiamas added a comment -

            Good idea ... this would simplify things a lot. Not sure how it will work out code-wise.

            Show
            dougiamas Martin Dougiamas added a comment - Good idea ... this would simplify things a lot. Not sure how it will work out code-wise.
            Hide
            dougiamas Martin Dougiamas added a comment -

            I'll have a shot at this one actually

            Show
            dougiamas Martin Dougiamas added a comment - I'll have a shot at this one actually
            Hide
            dougiamas Martin Dougiamas added a comment -

            This screenshot shows what I've got working here locally. I want to be sure people are OK with it before I check in.

            This one is for the module editing screen. The same tab names (and lack of second level tabs) appears on all other other role assignment pages too.

            Show
            dougiamas Martin Dougiamas added a comment - This screenshot shows what I've got working here locally. I want to be sure people are OK with it before I check in. This one is for the module editing screen. The same tab names (and lack of second level tabs) appears on all other other role assignment pages too.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            +1 Looks really simpler!

            Also, I hope the "extratabs" hack will be respected with this change ( http://tracker.moodle.org/browse/MDL-9736 )

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - +1 Looks really simpler! Also, I hope the "extratabs" hack will be respected with this change ( http://tracker.moodle.org/browse/MDL-9736 ) Ciao
            Hide
            jisner John Isner added a comment -

            I think it's an improvement. But don't change terminology "Override" to "Modify permissions" unless you make the change everywhere, including all documentation. Otherwise it will be an endless source of confusion, similar to the confusion that resulted when Inherit changed to Not set. IMO override is already deeply ingrained and should be retained.

            Show
            jisner John Isner added a comment - I think it's an improvement. But don't change terminology "Override" to "Modify permissions" unless you make the change everywhere, including all documentation. Otherwise it will be an endless source of confusion, similar to the confusion that resulted when Inherit changed to Not set. IMO override is already deeply ingrained and should be retained.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Yes, I was worrying about that too, but on the other hand the new string seems a lot better for newbies. Since the content under the tabs is the same I was assuming people used to 1.8 would understand the switch pretty quickly. Thoughts?

            Otherwise: "Override permissions" ?

            Show
            dougiamas Martin Dougiamas added a comment - Yes, I was worrying about that too, but on the other hand the new string seems a lot better for newbies. Since the content under the tabs is the same I was assuming people used to 1.8 would understand the switch pretty quickly. Thoughts? Otherwise: "Override permissions" ?
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Perhaps we could perform that in 2 steps:

            • "Override permissions" for 1.9. It's better for newbies and understandable for traditional users.
            • "Change permissions" for 2.0. Easy for both collectives too. IMO.

            Noting such changes in sensible places, of course (Docs, help files, forums...)

            +1 for that.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Perhaps we could perform that in 2 steps: "Override permissions" for 1.9. It's better for newbies and understandable for traditional users. "Change permissions" for 2.0. Easy for both collectives too. IMO. Noting such changes in sensible places, of course (Docs, help files, forums...) +1 for that. Ciao
            Hide
            dougiamas Martin Dougiamas added a comment -

            OK, I've checked in these changes (with "Override permissions"). I think it's a good step forward. There are still lots more we can do I'm sure.

            Show
            dougiamas Martin Dougiamas added a comment - OK, I've checked in these changes (with "Override permissions"). I think it's a good step forward. There are still lots more we can do I'm sure.
            Hide
            tsala Helen Foster added a comment -

            Override roles documentation page renamed as Override permissions, with new screenshot and note about page renaming.

            http://docs.moodle.org/en/Override_permissions

            Show
            tsala Helen Foster added a comment - Override roles documentation page renamed as Override permissions, with new screenshot and note about page renaming. http://docs.moodle.org/en/Override_permissions

              People

              • Votes:
                5 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Mar/08