The edit link is using a custom made dropdown that allows you to quickly jump to settings. I would advise using the D8 dropbutton for this, its used in operations and thoroughly tested on usability, a11y and more. Lending form that will make this more usable.
2013-09-28_1556.png

Comments

drunken monkey’s picture

Status: Active » Needs review
StatusFileSize
new6.59 KB

I definitely planned to do this in the D8 version, but doing it in D7 is a bit more difficult, of course. However, I guess we really should do this – and since the dropbutton is included in CTools, which most sites will probably have installed, we can maybe just use that and implement a very basic fallback in case it's not there after all.
(Or do you think there are enough non-CTools users out there to justify leaving the old functionality in there as the workaround?)

Patch attached. I'm no designer or themer, as is probably apparent, so the look might still not be optimal.

Bojhan’s picture

I guess its a little wierd to depend on CTools for something this small, but CTools is like a misc module anyway. So I think thats fine. The dependancy wasn't triggerd on Simplytest.me so it looks kinda wierd, without CTools enabled.

Looking at it with CTools enabled, it still looks and behaves quite weird (although less weird than before). I'd still argue to just copy over the D8 dropbutton, since that was heavily tested and designed.

drunken monkey’s picture

StatusFileSize
new21.05 KB

Makes sense, I guess. It just adds so much code to do this one little thing that I didn't want to do that.
However, I guess it's better to do it the right way, if at all, so attached is a patch porting D8's dropbutton for our overview page.

drunken monkey’s picture

See also: #2122155: Add the "Files" tab to contextual links, for using this for Solr servers.
(My new patch also now unifies all the operations in the dropbutton, for both indexes and servers. I think that makes more sense than arbitrarily including some in the dropbutton and others not.)

drunken monkey’s picture

Issue summary: View changes
StatusFileSize
new20.96 KB

Re-roll, due to #2100199: Consider merging view, edit, settings being committed.

Status: Needs review » Needs work

The last submitted patch, 5: 2100193-5--drobuttons.patch, failed testing.

drunken monkey’s picture

Version: 7.x-1.8 » 7.x-1.x-dev
Status: Needs work » Needs review

Wrong version.
(What use is testing if the patch applies on an old version?!?)

drunken monkey’s picture

5: 2100193-5--drobuttons.patch queued for re-testing.

drunken monkey’s picture

Status: Needs review » Fixed

Committed.
Thanks again!

Status: Fixed » Closed (fixed)

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