Opened 16 months ago
Last modified 16 months ago
#56403 reviewing defect (bug)
Write 'Activated' instead of 'Installed' for activated theme
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Themes | Keywords: | reporter-feedback has-patch |
Focuses: | javascript | Cc: |
Description
On theme-install page, there have text for all installed theme but it should be 'Activated' instead of 'Installed' for activated theme.
Attachments (6)
Change History (11)
#1
@
16 months ago
- Component changed from Text Changes to Themes
- Owner set to audrasjb
- Status changed from new to reviewing
- Type changed from enhancement to defect (bug)
- Version trunk deleted
#2
@
16 months ago
- Keywords reporter-feedback added
Hello @alireyad welcome to WordPress Core Trac and thank you for opening this ticket,
I may be misunderstanding the issue, but the current implementation looks good to me. There's a difference between:
- Installed themes (may be several items)
- Installed AND activated theme (only one item).
#3
@
16 months ago
- Keywords has-patch added
Alright, but I have some concerns about duplicating the "Active/Activated" mention. See screenshot below.
Please note that the patch contains some unrelated file changes (but it can eventually be cleaned up before commit).
#4
@
16 months ago
Hi there, welcome to WordPress Trac! Thanks for the ticket!
I think I agree with @audrasjb here. For the purpose of the Add Themes screen, the relevant information to know is which themes are already installed, and the activated theme is already marked as such next to the theme name.
So the current implementation makes sense to me and I'm not sure if it needs any changes.
Current implementation