Opened 14 years ago
Closed 13 years ago
#17212 closed defect (bug) (fixed)
Within the ThickBox gallery, the .pinkynail image's display toggling behavior is weird.
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.3 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Gallery | Keywords: | has-patch |
| Focuses: | Cc: |
Description
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.
Attachments (1)
Change History (8)
#2
follow-up:
↓ 3
@
14 years ago
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)!
#3
in reply to:
↑ 2
;
follow-up:
↓ 4
@
14 years ago
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)?
#4
in reply to:
↑ 3
@
14 years ago
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.
By the way, the patch is for the file: /wp-admin/js/gallery.dev.js