![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 12 years ago
Closed 12 years ago
#27640 closed defect (bug) (fixed)
Theme Install page does not display errors
| Reported by: | johnbillion | Owned by: | matveb |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Themes | Version: | 3.9 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | ui |
Description
If the AJAX call to api.wordpress.org on the theme install screen fails, the error message which is supposed to be shown via the query:fail event does not show up.
Looks like the issue is due to the display:none here in themes.css.
Attachments (4)
Change History (11)
#2
@
12 years ago
- Keywords has-patch added; needs-patch removed
27640.patch fixes it for me.
div.themes is removed to avoid a 100px gap between the menu and the error message. From my testing, it is reinserted after a successful AJAX call.
#3
@
12 years ago
Actually, that gap (added along with the file in [26142]) doesn't seem to make a difference anywhere else. 27640.2.patch removes it from CSS.
#4
@
12 years ago
One more take in 27640.3.patch, just for good measure.
Note:
See TracTickets
for help on using tickets.
Confirmed. The spinner also never disappears in that case.