Problem/Motivation

Admin language negotiation is not functional if placed below path prefix negotiation with fallback if visiting the admin page in a path prefix for example. It makes most sense to move up this negotiation method so when enabled, it makes sense where it is.

Proposed resolution

Move the admin language negotiation up to be the first option.

Remaining tasks

Do it.

User interface changes

Admin language negotiation will be the first option but not enabled by default. (The second option will be path negotiation still enabled by default. No change there.)

API changes

None.

Comments

penyaskito’s picture

Issue summary: View changes

After #1998648: "Administration pages language" user account setting should be hidden if not used landed, I guess there is no usability problem anymore.

I don't feel as enabling "admin language negotiation" by default because I never had this need and can be confusing for a user having two languages at his profile.

However, as I found some related issues that became quite controversial, I'm not closing this as won't fix (but I would like to).

gábor hojtsy’s picture

Title: Move admin language negotiation up to first option and enable it by default » Move admin language negotiation up to first option
Issue summary: View changes
Issue tags: +Novice

Following #1998648: "Administration pages language" user account setting should be hidden if not used we should not enable this by default IMHO. But moving up to first would be useful because otherwise it will not take effect. Updated summary, marking novice.

bogdan.racz’s picture

Assigned: Unassigned » bogdan.racz
bogdan.racz’s picture

Changed the default value.
Added the patch, please review.

bogdan.racz’s picture

Assigned: bogdan.racz » Unassigned
Status: Active » Needs review
gábor hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Makes sense. I don't think this is worth testing (with automated tests).

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Usability changes are prioritised in beta and this one is not disruptive. Committed 8d5ea64 and pushed to 8.0.x. Thanks!

  • alexpott committed 8d5ea64 on 8.0.x
    Issue #1833012 by rbmboogie: Move admin language negotiation up to first...
tstoeckler’s picture

Issue tags: +drupaldevdays

Status: Fixed » Closed (fixed)

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