[wp-trac] [WordPress Trac] #27055: Themes Install Screen: introduce THX (Backbone powered) UX
WordPress Trac
noreply at wordpress.org
Tue Mar 11 03:54:26 UTC 2014
#27055: Themes Install Screen: introduce THX (Backbone powered) UX
-------------------------+------------------------------
Reporter: matveb | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Themes | Version: trunk
Severity: normal | Resolution:
Keywords: | Focuses: administration
-------------------------+------------------------------
Comment (by gcorne):
@matveb, I really enjoy the improved design!
Took a quick pass in [attachment:27055.9.diff 27055.9.diff] to tighten up
the javascript and make a few improvements.
* Removed the unused 'tag' field from the query. Doing so dramatically
sped up the API calls (going from 2-7 seconds to more like 280 ms).
* Switched to https for the client-side API calls
* Switched to inheritance, new objects, or lightly adding new methods to
existing constructors instead of overwriting methods/properties on-the-fly
* Fixed issue where infinite scroll wasn't working in some cases such as
the "latest" and "popular" filters
* Made a jshint pass.
* Fixed up the router a bit to be more inline with how themes.php works.
While making this pass, I came up with some things that I think still need
to be done:
* Use the same treatment for themes that are already installed as
{{{themes.php}}}
* Finish up the "More" filters
* Carefully think through what data coming from the API needs to be
escaped.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/27055#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list