All popup IFRAMEs are created with attribute scrolling="no". If the content inside the IFRAME is "bigger" than the actual modal view (in it's maximum size), user should be able to scroll the content. This works properly in Firefox for instance but Chrome ignores the overflow-y:scroll CSS and disables scrolling of the IFRAME. There are possible CSS fixes available but none work nicely in all browsers.

Suggested fix is to set scrolling attribute to 'auto' as it's the default state and only explicitly set to 'no'.

Files: 
CommentFileSizeAuthor
#9 without_patch.png264.39 KBDevin Carlson
#9 with_patch.png266.63 KBDevin Carlson
#1 media-scrolling-issue-1359524-2.patch580 bytespeterpajchl
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

StatusFileSize
new580 bytes
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Enclosed patch sets IFRAME scrolling attribute defaults to 'auto'.

Status:Active» Needs review

Setting to "needs review"

Version:7.x-2.0-unstable2» 7.x-2.x-dev

Still relevant?

#1: media-scrolling-issue-1359524-2.patch queued for re-testing.

@DaveReid or somebody who is in the know:

Do we still need this?

Yes, it's still relevant. I have 2 rows of items in my View and can't see the submit button without holding the middle mouse button down and dragging.

You can speed up the process if you could test the patch and post your findings here. ;)

I'm having a problem duplicating this condition in Chrome. If I increased the pager limit for the Media browser view to 50, the dialog still sizes it's height correctly so that I don't have to scroll.

Status:Needs review» Reviewed & tested by the community
StatusFileSize
new266.63 KB
new264.39 KB

I've tested this with a number of themes and, depending on how they style things like exposed views filters, the buttons can be pushed off of the page and require a middle mouse button click to scroll to.

This patch guarantees that users will still be able to use the buttons regardless of which administrative theme they are using. It would also enable the extra "scrollbox" div on the library tab to be removed.

The patch applied cleanly with an offset. I've attached an image showing how the buttons can be hidden (using bartik as an example) and how the patch enables the iframe to be scrolled so the buttons can be reached.

I think that this is the best option, short of removing the ability to select a custom administration theme.

tested the patch, the above bug is not a duplicate

Status:Reviewed & tested by the community» Fixed

Status:Fixed» Closed (fixed)

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