In coherence with #546956: [meta-issue] Overhaul of Information Architecture the creation of a User Management category. Also named 'People' , this category serves as goal to be a house for Permission, Roles, User settings and any other setting that could be considered only related to Management of users.

It is inevitably that this category is required, without it finding anything related to roles, permissions or user settings is nearly impossible.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yoroy’s picture

People stuff

  • Roles
  • Permissions
  • Account setup
    • (Account setup is my rename of the rather empty 'user settings', but don't let that distract you)

yoroy’s picture

It's such a small list, you can't help but think it could be put on the People page itself…
We need to connect the People page with this group of settings on the Config page somehow (this is also slightly off topic)

The topic is:

- label for this People Management (ugh) category
- items that will certainly live there

webchick’s picture

Don't forget about profile fields to user profiles. I don't think that belongs coupled with people management. OTOH, configuring permissions is something you do on a minutely basis when setting up the site, and it might make sense for roles/permissions to be a lot closer to the people management screen.

alexanderpas’s picture

That category should be created, and i think it should be something like this:

People Management
* People Listing
* Account Settings
* Profile Fields
* Roles
* Permissions

(note: bikeshedding over names etc. can be done later on.)

also: a category is very important, as in the current situation, the people link only shows in the management menu at the bottom of the page when Seven is set ad admin menu, and the expert profile has been used.

Bojhan’s picture

Assigned: Unassigned » Bojhan

-

yoroy’s picture

Ok, summary of IRC chat with Bojhan:

For the category label:

- Let's avoid 'People management', it's corporate speak at it's worst and will make catch quit Drupal. Can't have that.
- 'Management' is rather meaningless anyway. We're in the context of the admin of a content management system, it's all about managing stuff so duh.

People & permissions is what we came up with, for the following reasons:

- 'Permissions' is a key word. In this category, it'll probably be the page used most often.
- Bojhan mentioned that during the Baltimore usability tests, 'Permissions' was a care word indeed. "people think everything (that goes wrong) is about permissions."
- Bojhan checked what Basecamp does and lo! they're using 'People & Permissions'…
- 'People and permissions' implies a pretty broad scope on what could be underneath it (more so than say, 'People & roles', yet is still quite distinctive from the other categories.

webchick’s picture

I support the direction in #6. Since we call the top-level link "People" it's natural people will be looking for that keyword in the settings page as well, and I can also confirm that "permissions" is a critical keyword, based both on formal usability testing and also my general observations in teaching workshops, etc. There's a good reason we renamed "Access control" to "Permissions" in D6. :)

catch’s picture

I like 'People and permissions' - it opens this section up to more properly deal with non-user-account-holding visitors, permissions incorporates roles (at least in Drupal), doesn't make me want to cry or punch walls like 'People management' does.

alexanderpas’s picture

People & permissions
that is one Great Find, and in addition, it makes the other one (config & modules) look less weird.

catch’s picture

Title: Configuration page: People Management category » Configuration page: People & permissions
Gábor Hojtsy’s picture

Status: Active » Needs review
FileSize
29.74 KB
39.29 KB

Moved all mentioned user settings and profiles to "People and permissions". This is how it looks:

I've put this up as the first item on the right to list. Probably these left/right placements and weights will be debated after code freeze, but we need the paths to settle, so let's not argue over that. Also, I've put the order of items to be "Account settings", "Roles", "Permissions" first in a logical order that my mind maps. That is rather easy to fix afterwards.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

It worked for me, and seems no one debated the label - which is very straight forward. Marking this RTBC so we can move on, and actually provide some good IA before code freeze.

Bojhan’s picture

Assigned: Bojhan » Unassigned

No idea, why this was marked to me. Would love to see someone validate this RTBC.

yoroy’s picture

Status: Reviewed & tested by the community » Needs work

IP adress blocking seems to fit in here as well but isn't part of it yet.

webchick’s picture

Status: Needs work » Fixed

I went to commit this, but it looks like Dries already did. :)

Bojhan’s picture

Issue tags: +Needs documentation

*cough*

webchick’s picture

Status: Fixed » Needs work
Issue tags: -Needs documentation

Oops. But yes, let's move IP blocking here.

webchick’s picture

Issue tags: +Needs documentation

Hahhaa

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
10.04 KB

Patch to move IP blocking.

Status: Needs review » Needs work

The last submitted patch failed testing.

jromine’s picture

Priority: Normal » Critical
Status: Needs work » Needs review
FileSize
737 bytes

Editing profile fields broken, e.g., admin/config/people/profile/edit/1
patch attached

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good. Unfortunate that we don't have tests for this :( We should come back to the ip blocking move after this fix is committed.

Dries’s picture

Committed both patches. Thanks!

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed both patches. Thanks!

Dries’s picture

Status: Fixed » Needs work

#19 didn't actually passed so rolled that back.

jromine’s picture

Status: Needs work » Needs review
FileSize
10.03 KB

fixed ip-blocking patch

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Changing arg numbers is something we happen to forget. Thanks for fixing that up. Looks good, passed UX already above, passed tests -> RTBC.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Usability, -Needs documentation, -IA, -D7UX

Automatically closed -- issue fixed for 2 weeks with no activity.