Media page jumps after unfolding image details

[caesarsgrunt]

On the Media Library and Gallery pages, when you click Show next to an image, the following happens :

1. The image details slide down.
   
2. The page jumps so that the row for the image in question is at the top of the page.

Step 2 should not happen when the whole image details box fits on the page anyway, as it causes an unnecessary jump.

It should only scroll when it will help the image details to fit on the page better.

[caesarsgrunt]

Fixes the scrolling issue.

[scribu]

Tested in FF 3.5.

[caesarsgrunt]

Is anyone going to commit this bugfix?

[scribu]

Probably. 2.9 isn't out yet, so there's still hope.

[janeforshort]

I've asked Andrew to take a look, but for the record, saying "Tested in FF 3.5" is in no way sufficient to justify a commit. Please rally some community members to test it on additional platforms etc.

[caesarsgrunt]

Jane, I know scribu only said he tested it in FF 3.5 - but myself, if I recall correctly, I tested it in Firefox 3.5, Safari 4, Opera 10, and IE 8 before uploading the patch. I do this for most patches to client-side code.

[azaozz]

(In [12349]) Do not auto-scroll the media popup when opening an image details section if it can fit in the viewport, props caesarsgrunt, fixes #11269