![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 13 years ago
#27695 closed defect (bug) (fixed)
Theme Installer: already installed themes should indicate 'Installed'
| Reported by: | adamsilverstein | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Themes | Version: | 3.9 |
| Severity: | normal | Keywords: | has-patch dev-feedback |
| Cc: | Focuses: | javascript |
Description
Currently when browsing the new theme installer, there is no indication that a theme is already installed. In both the browse and detail views, themes show an install button.
Instead we should at a minimum show that the theme is 'Installed'. Clicking the Install button in the browser actually opens the detailed view (another ticket for that), clicking the install button in the detail view on an already activated theme leads to an error.
Attachments (1)
Change History (6)
#2
@
13 years ago
27695.diff Show Installed for installed themes on the theme installer browser and detail view.
#3
follow-up:
↓ 4
@
13 years ago
- Milestone Awaiting Review → 3.9
There is already another patch by nacin to solve this issue: https://core.trac.wordpress.org/attachment/ticket/27055/27055.34.diff
#4
in reply to: ↑ 3
@
13 years ago
Replying to ocean90:
There is already another patch by nacin to solve this issue: https://core.trac.wordpress.org/attachment/ticket/27055/27055.34.diff
Ah, thanks for pointing that out. I searched trac before submitting the ticket and didn't find that patch. Both approaches seem reasonable, I will leave it up to the leads to decide how to proceed.
show 'Installed' on installed themes in theme installer