#34206 closed task (blessed) (fixed)
Add a Favorites Tab to the Add New Themes Page
Reported by: | jeffr0 | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Themes | Keywords: | has-screenshots has-patch |
Focuses: | ui | Cc: |
Description
The ability to mark themes as favorites in the WordPress theme directory was added three months ago https://meta.trac.wordpress.org/changeset/1770 Unlike the Add New Plugins page in the backend of WordPress, the Add Themes page does not have a Favorites tab to browse and install themes marked as Favorites. For consistency, I think Favorites should be added to the Add Themes page.
This screenshot is the Add Themes page in WordPress 4.4 Alpha
https://cloudup.com/cN6rYpDiPUw
Attachments (6)
Change History (25)
#2
@
9 years ago
- Keywords has-patch needs-refresh added; needs-patch removed
34206.diff adds the "Favorites" tab to the theme installer.
The only thing missing is the form to enter your username if absent, like on the plugin install screen. Perhaps it doesn't make sense to have this form at two places though.
#5
@
9 years ago
- Keywords needs-refresh removed
@swissspidy We're going to need the username field for this to be ready. Did you want to take a stab at it? 34206.diff still applies.
#7
@
9 years ago
- Keywords dev-feedback added
34206.2.diff is a new patch that adds the username form as well.
Data is submitted via GET, just like on the plugin install screen. Because of that I needed to add a sort.split('&')[0]
line to the sort method. Otherwise it would try to sort themes after favorites&user=<user>
. Haven't found the proper way in the router to change that. It's not a big deal, but maybe someone wants to make this more beautiful.
This ticket was mentioned in Slack in #core by helen. View the logs.
9 years ago
#11
@
9 years ago
- Milestone changed from Future Release to 4.4
- Type changed from enhancement to task (blessed)
#12
follow-up:
↓ 13
@
9 years ago
- Keywords needs-patch added; has-patch dev-feedback removed
Tested 34206.2.diff:
- If I open the Favorites tab and then click on any other tab, the username input form does not disappear, because the
.show-favorites-form
class is never removed. - Any chance to get rid of the page refresh when clicking Get Favorites? It could probably work the same way as the Apply Filters button on Feature Filter tab.
#13
in reply to:
↑ 12
@
9 years ago
Replying to SergeyBiryukov:
- If I open the Favorites tab and then click on any other tab, the username input form does not disappear, because the
.show-favorites-form
class is never removed.
Argh, I could swear I've tested this.
- Any chance to get rid of the page refresh when clicking Get Favorites? It could probably work the same way as the Apply Filters button on Feature Filter tab.
That's what I first tried to implement, but I encountered problems with the request cache. Will look into it again though.
#14
@
9 years ago
- Keywords has-patch added; needs-patch removed
34206.3.diff is an updated patch that fixes the bug mentioned by @SergeyBiryukov and also uses Ajax to save the username.
Saving happens when clicking on the button or pressing enter. When saving was successful, a second query to browse the themes happens.
+1 for this. According to the
themes_api
docs querying favorites should already be possible.