Opened 17 months ago
Last modified 11 days ago
#54839 new defect (bug)
Image upload after dirty filter requires filter change before it can be inserted to page
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | needs-patch has-testing-info |
Focuses: | ui, javascript | Cc: |
Description
Steps:
- Open media library popup.
- Change date filter.
- Upload new image.
- See uploaded image is not visible in list nor selectable. (Select button is disabled)
- Change filter to match the new image uploaded, See image is visible.
Expected behavior:
- Uploaded image should be selectable if it matches filter (or should reset filter on tab change).
Current behavior:
- Uploaded image after dirty filter is not selectable.
Attachments (1)
Change History (17)
#1
@
17 months ago
- Milestone changed from Awaiting Review to 6.0
- Version trunk deleted
Hello @utkarshpatel,
Welcome back to WordPress Core's Trac 👋 Thank you for reporting this issue!
I can reproduce the issue on trunk
, 5.8.3, and 5.5.8 (stopped testing at this point release). Investigation will be needed to identify when the bug was introduced and root cause. As I can reproduce, pulling it into the 6.0 milestone for further investigation.
As this is not due to 5.9, removing trunk
as the Version. Once the version is identified, please update.
#2
@
16 months ago
Hello @hellofromTonya,
Thank you!
I was able to reproduce the issue for version 5.0 - 5.4
. This was introduced in 5.0
and 4.9
handles this correctly.
I will work on the patch.
This ticket was mentioned in Slack in #core by costdev. View the logs.
14 months ago
#5
@
14 months ago
- Keywords needs-patch has-testing-info added
- Milestone changed from 6.0 to 6.1
- Summary changed from Image upload after dirty filter requires filter change before it cab be inserted to page to Image upload after dirty filter requires filter change before it can be inserted to page
After the discussion in the bug scrub, I'm moving this ticket to the 6.1 milestone, adding the needs-patch
and has-testing-info
keywords, and correcting a minor typo in the ticket title.
This ticket was mentioned in Slack in #core-media by antpb. View the logs.
10 months ago
This ticket was mentioned in Slack in #core-media by antpb. View the logs.
9 months ago
#8
@
9 months ago
- Milestone changed from 6.1 to 6.2
This one still needs a patch so might be better suited for 6.2. Moving to the next milestone but feel free to move it back if anyone has the bandwidth to make it happen for 6.1!
#9
@
4 months ago
This is a tough one, the backbone app is quite hard to navigate, @adamsilverstein might be able to give some guidance here.
I tried to investigate but I'm a bit rusty inside the Media Library code to be honest.
Screencast demonstrating bug