#29367 closed defect (bug) (fixed)
Media Modal: the progress bar for each attachment no longer shows upload progress
Reported by: | gcorne | Owned by: | azaozz |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | 4.0 |
Component: | Media | Keywords: | has-patch |
Focuses: | javascript | Cc: |
Description (last modified by )
The progress bars in the attachment browser no longer animate when uploading. See https://cloudup.com/cAGpTEdfq6A for a demonstration. The progress bars worked correctly in 3.9.2.
This broke in [29266] because rerenderOnModelChange
is set to true
so any change triggers the view to re-render. The upload progress isn't handled during a normal Attachment view render, so always re-rendering causes any change to the progress bar to be overwritten.
Attachments (1)
Change History (8)
#1
@
10 years ago
- Component changed from General to Media
- Description modified (diff)
- Focuses javascript added
- Version set to trunk
This ticket was mentioned in IRC in #wordpress-dev by gcorne. View the logs.
10 years ago
#5
@
10 years ago
- Keywords has-patch added
29367-01.patch fixes the issue by incorporating the percentage of progress into the template that is used to render. So when the change event is firing a re-render, the current percentage is reflected in the width of the progress bar. I handled it this way instead of calling this.progress()
after this.$el.html()
so that the progress bar wouldn't flicker.
Actually, it looks like the binding of
render
tochange
happened in [28993].