#29367 closed defect (bug) (fixed)
Media Modal: the progress bar for each attachment no longer shows upload progress
| Reported by: | gcorne | Owned by: | azaozz |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.0 |
| Component: | Media | Version: | 4.0 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | javascript |
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
@
12 years ago
- Component General → Media
- Description modified (diff)
- Focuses javascript added
- Version → trunk
This ticket was mentioned in IRC in #wordpress-dev by gcorne. View the logs.
12 years ago
#5
@
12 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Actually, it looks like the binding of
rendertochangehappened in [28993].