Within the ThickBox gallery, the .pinkynail image's display toggling behavior is weird.

[goto10]

Different clicking sequences between the individual and all show/hide controls will cause the pinkynail to be either hidden when it should be shown, or vice versa.

To reproduce the issue:

1.) Go into a post, and click the "Add Image" button to launch the ThickBox iframe.
2.) Either upload some new images or use a post that already has some images attached.
3.) Click the "Gallery" tab.
4.) Click the "Show" link for one single media item.
5.) Click the "All Tabs: Show" link
6.) Finally, click the "All Tabs: Hide" link.

The pinkynail image for the individual media item clicked in step 4 will not be toggled back into display.

This simple patch forces the pinkynail to be hidden when showing all, and shown when hiding all.

By the way, the patch is for the file: /wp-admin/js/gallery.dev.js

I saw you mentioned this was your first patch in the #wordpress-dev channel, but when you run svn diff to generate a diff file, you can run it from the root of the install to automatically show the full path of the file being edited. Also works better when editing multiple files. Just a heads up if you keep contributing (which you should)!

Replying to andrewryno:

I saw you mentioned this was your first patch in the #wordpress-dev channel, but when you run svn diff to generate a diff file, you can run it from the root of the install to automatically show the full path of the file being edited. Also works better when editing multiple files. Just a heads up if you keep contributing (which you should)!

Thanks andrewwryno. Yes indeed, this is my first contribution, so I appreciate the advice. Should I create a new patch from the root, or is it OK as is (and just go from root in the future)?

Replying to goto10:

Replying to andrewryno:
Thanks andrewwryno. Yes indeed, this is my first contribution, so I appreciate the advice. Should I create a new patch from the root, or is it OK as is (and just go from root in the future)?

Commiters/testers prefer from root, but it's a small patch so I think it's fine.

Replaced previous patch with one created from the root directory

In [18851]:

Toggle the pinkynails properly, props goto10, fixes #17212