skinr_ui module adds skin editing contextual links to body element. This makes contextual links to appear in a wrong way:

  1. Even when a mouse is not hovering any region, all the contextual links are visible
  2. When using Overlay module, underlying links visible through the overlay- see the screenshot.

Providing simple patch, which skips processing of contextual links for 'html' hook.

CONS This way, we are loosing a way to edit system_block skin - we just missing it.

CommentFileSizeAuthor
dont_add_contextual_links_to_body_element.patch601 bytesOnkelTem
Screenshot from 2012-08-08 20:44:46.png68.74 KBOnkelTem

Comments

moonray’s picture

moonray commented
Status: Needs review » Closed (duplicate)

Duplicate of #1588158: admin menu showing at bottom

OnkelTem’s picture

OnkelTem commented

@moonray

I don't see how it is resolved there. Can you explain here why this is dup?

moonray’s picture

moonray commented

It's exactly the same issue. We've discussed the pros and cons of approaches there, and come up with a possible solution (even though it's not yet implemented).

OnkelTem’s picture

OnkelTem commented

@moonray

Ok, I'm gonna read that discussion.