I apologize for not having the correct terminology to describe this -- I'm not a themer.

The <div id="toolbar-administration"> element sits to the right of the visual location of that element when the toolbar is docked on the left of the viewport. The result is you are unable to click on elements under this ghost

since it's higher in the z-index than the table. Hopefully this screenshot will convey what I'm not able to explain well...

Note: this requires the toolbar to be on the left-side of the screen (works correctly when docked at the top) and only seems to be an issue on the admin/content/files page.

Sorry for the lousy bug report -- only had a few minutes.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

idebr’s picture

Issue tags: +frontend, +CSS, +Usability

@mikeker Thanks for submitting this bug report! I was able to reproduce this issue on /admin/content/files.

After an initial investigation the issue seems to be caused by the contextual links region styling on the body element that exists on the Files administration page, but not on the Content administration page.

idebr’s picture

Component: Seven theme » toolbar.module
Status: Active » Needs review
Issue tags: +Needs manual testing
FileSize
607 bytes

This appears to be an issue with the Toolbar rather than Seven. This CSS was introduced in #1860434: Refactor the Toolbar JavaScript to use Backbone; fix several poorly functioning behaviors comment #48 without much explanation, so needs manual testing in different browsers to test nothing broke.

mikeker’s picture

Verified it fixes the issue reported -- I haven't done any testing beyond that so I'm not moving it to RTBC. I'll leave that to someone that better understands front-end CSS... :)

Thanks, @idebr, for finding the root cause!

LewisNyman’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs manual testing

I had a look over the CSS to find the intentions of the code and I couldn't see a good reason for it. I also tested the patch in multiple browsers with no regression. Thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Needs review

So this only is happening on admin/content/files and not admin/content or admin/content/comment - how come? I think something else is wrong.

LewisNyman’s picture

Issue summary: View changes
Status: Needs review » Needs work
FileSize
696.61 KB

@alexpott Of course! You're right, on the content/files page 'position: relative' is added to the body tag which is not good...

idebr’s picture

Status: Needs work » Needs review

Re: @alexpott in #5

So this only is happening on admin/content/files and not admin/content or admin/content/comment - how come? I think something else is wrong.

Views has an advanced option 'Contextual links' that is enabled for admin/content/files but not for admin/content. While inconsistent, either option is valid.

Re: @Lewis Nyman in #6

@alexpott Of course! You're right, on the content/files page 'position: relative' is added to the body tag which is not good...

Can you elaborate what is 'not good' about it? It's perfectly valid CSS.

LewisNyman’s picture

Can you elaborate what is 'not good' about it? It's perfectly valid CSS.

Right but if you add this kind of of styling to the body, it's impossible to know if you will be introducing regressions because it's not possible to insert HTML outside of the body tag. I don't think we should every style the body tag directly. Any chance we can add it to a wrapper div or something?

idebr’s picture

@LewisNyman There is an issue in the contextual.module component regarding contextual links for the main page content #1916516: Decide whether/how to implement contextual links associated with the main page content. Does this cover your concern or am I missing your point entirely? :)

LewisNyman’s picture

Yeah it seems like it could solve the problem, but not anytime soon :\

jhedstrom’s picture

This needs an IS update with remaining tasks. The patch still applies, but doesn't seem to be the agreed-upon resolution.

malcomio’s picture

Status: Needs review » Fixed
FileSize
542.6 KB

I can't reproduce this issue - I assume it must have been fixed elsewhere.

Status: Fixed » Closed (fixed)

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