Our default facet controller is “Edit” at this moment, while the “Display” tab is the route most people will be using when configuring facets.

I suggest we follow core’s Field UI flow, thus:
rename our “Display” route to “Edit”, and use this as main route.
rename our current “Edit” route to “Facet settings”, and make it the 2nd tab instead.

In the facet overview page, we can still show the “edit” button this way, but it will link to the page most users actualy need (instead of having to use the dropdown).

CommentFileSizeAuthor
#9 change_admin_controller-2712511-9.patch86.19 KBStryKaizer
#6 change_admin_controller-2712511-6.patch82.7 KBStryKaizer
#3 change_admin_controller-2712511-3.patch6.41 KBStryKaizer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StryKaizer created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Component: Code » User interface

I agree, this another great UX improvement. Do you think we can tag this as novice as well?

StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
Status: Active » Needs review
FileSize
change_admin_controller-2712511-3.patch6.41 KB

Tests still running, need to fix a lot I guess ;)

Status: Needs review » Needs work

The last submitted patch, 3: change_admin_controller-2712511-3.patch, failed testing.

The last submitted patch, 3: change_admin_controller-2712511-3.patch, failed testing.

StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
Status: Needs work » Needs review
FileSize
change_admin_controller-2712511-6.patch82.7 KB

Status: Needs review » Needs work

The last submitted patch, 6: change_admin_controller-2712511-6.patch, failed testing.

The last submitted patch, 6: change_admin_controller-2712511-6.patch, failed testing.

StryKaizer’s picture

StryKaizer
Primary language Dutch
Location Belgium
CreditAttribution: StryKaizer as a volunteer and commented
Status: Needs work » Needs review
FileSize
change_admin_controller-2712511-9.patch86.19 KB

  • borisson_ committed c661eb2 on 8.x-1.x authored by StryKaizer 
    Issue #2712511 by StryKaizer: Change admin controller routing for better...
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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