Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.There is currently a clear distinction between; viewing the index/server, managing the the index/server, and its settings. I think this could all be merged, now you are easily lost amongst the 7 configuration tabs.
1) Remove, move most of the settings from View to Settings.
2) Put the indexing below on the settings page.
Delete could probably also just be a button on the edit page, no need for it to be a tab.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 2100199-13--merge_tabs.patch | 25.85 KB | drunken monkey |
| #8 | 2100199-8--merge_tabs.patch | 25.71 KB | drunken monkey |
| #5 | 2100199-5--merge_tabs.patch | 22.3 KB | drunken monkey |
| #5 | screen1.png | 37 KB | drunken monkey |
| #5 | screen2.png | 35.58 KB | drunken monkey |
Comments
Comment #1
Bojhan CreditAttribution: Bojhan commentedMarking major because this could greatly enhance the UX. It reduces the N of tabs by 3.
Comment #2
drunken monkeyThanks for the suggestion, cleaning up there a bit does of course make sense.
However, I'm not sure how to make the resulting combined tab clean and well-structured enough that this would really result in an improvement.
How about combining just "View" and "Status" into a new "Status", and moving "Delete" to a button on the "Edit" form? That should be both a noticable improvement and still not swamp the remaining tabs with functionality.
Otherwise, how would you structure the resulting combined tab?
Comment #3
Bojhan CreditAttribution: Bojhan commentedSure, let me try and visualize this.
I would be oke with the interim step of merging View Status, and moving delete to the edit though. That would already be a big improvement.
Comment #4
Bojhan CreditAttribution: Bojhan commentedScratch that, lets just do your suggestion. Merging it with settings requires too much custom design.
Comment #5
drunken monkeySeeing as most of the data we display on "View" can be changed at "Edit", it might actually be possible and make sense to combine them … It just might be a bit confusing to users. Also, what little use the "Description" field had would be gone almost completely, since the only place its value would now show up is when editing it – which still gets the information there, I guess, but just looks and feels weird.
In any case, we can first just do the easier merges and maybe discuss later whether a complete merge would make sense. Attached is my attempt at this. I also merged "Re-indexing" and "Clear index", since they have largely the same functionality, and just added a checkbox for deciding between the two.
Comment #7
drunken monkeyNo idea what happened there with the title …
Comment #8
drunken monkeyAnd here again, with the tests adapted.
Comment #9
Bojhan CreditAttribution: Bojhan commentedReviewing tonight.
Comment #10
Bojhan CreditAttribution: Bojhan commentedHmmpf, this is really hard to test - how do I test using simplytest.me? I tried to add a patch to my local repo, but I did not get what is in the screens above.
Comment #11
drunken monkeyDid you clear the cache? The above patch works fine for me.
Otherwise, you can just use this link to test the patch with simplytest.me. You'll just have to install an additional module for the service class and create a server, otherwise the display on the index page will be restricted.
Comment #12
Bojhan CreditAttribution: Bojhan commentedLooked at it, I think this is a fine interim step to making this better. I really like your action on these issues!
It reduces the time to get to critical tasks, which is quite important.
Comment #13
drunken monkeySorry for leaving this lying around so long. I wanted to make sure everyone who wanted to chime in had the opportunity, all UI changes would be committed at once (so the UI would only change once) and that a new Search API would be released before that (so users seeing my Prague presentation would not be surprised by a different UI right at the beginning).
Attaching a necessary re-roll, if that passes I'll happily commit.
Thanks again for all your help in this!
Comment #14
drunken monkeyExcellent. Committed.