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).

Comments

StryKaizer created an issue. See original summary.

borisson_’s picture

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

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

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

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

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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