This patch hides operation links and shows them only on hover

CommentFileSizeAuthor
#6 link.png164.89 KBcatch
#6 no-link.png10.23 KBcatch
#2 hide_operations.patch20.73 KBdmitrig01
hide_operations.patch5.41 KBdmitrig01

Comments

dmitrig01’s picture

dmitrig01 commented

So far i've converted admin/content/node, admin/content/taxonomy and admin/user/user but i'm working on more.

dmitrig01’s picture

dmitrig01 commented
StatusFileSize
new hide_operations.patch20.73 KB

Converted
admin/build/block
admin/build/types
admin/build/menu
admin/build/menu-customize/*
admin/build/themes
admin/build/path
admin/settings/actions
admin/settings/ip-blocking
admin/settings/formats
admin/user/roles

and now I realize there is a better way to do it. Patch forthcoming

Status: Needs review » Needs work

The last submitted patch failed testing.

dries’s picture

dries commented
Status: Needs work » Needs review

Awesome. Might be some overlap with #492834: Hover operations for flooded state screens.

Bojhan’s picture

Bojhan commented
Issue tags: +Usability, +Needs screenshots

No screenshot, no tag. I can't review the overlap

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Needs work
StatusFileSize
new no-link.png10.23 KB
new link.png164.89 KB

Screenshots. At the moment this looks like a regression to me. The links are really jumpy (no delay when you move mouse away for example), and keeping the column there doesn't save any space - just looks empty. Also the thing which makes this look pretty in d7ux is the theming of the links, which isn't happening here - not that it necessarily has to be in one patch but would be good to know the route to it.

dmitrig01’s picture

dmitrig01 commented

Oh! I didn't know this was to save space - it looks really really jumpy when the column width changes. I thought it was to reduce clutter. ANd yes, at the moment they look ugly. I'm not sure how to fix that - I'm not a designer

catch’s picture

catch
he/him
Primary language English
commented

I think it's to save space, or at least that's what I thought - here's the 'find people' page with the delete appearing on hover - http://farm3.static.flickr.com/2478/3619353274_cfa94baf68_o.jpg

yched’s picture

yched commented

I'm not sure how the D7UX screenshot in #8 scales when there is more than just one 'remove' op ?

joachim’s picture

joachim commented

I don't understand the point of this at all.
Column looks empty without the links, because I am used to the existing UI; and if I were new to Drupal, how would I know there is something interesting there for me to use?
-1 from me.

Bojhan’s picture

Bojhan commented
Status: Needs work » Closed (won't fix)

To be clear for Drupal 7 this is a won't fix, and perhaps we will revisist this for Drupal 8. But as it is now, this adds no value.