[wp-trac] [WordPress Trac] #19815: Theme install and search screens should infinitely scroll
WordPress Trac
wp-trac at lists.automattic.com
Wed Feb 1 07:46:26 UTC 2012
#19815: Theme install and search screens should infinitely scroll
----------------------------+------------------
Reporter: helenyhou | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 3.4
Component: Themes | Version: 3.3
Severity: normal | Resolution:
Keywords: |
----------------------------+------------------
Comment (by DH-Shredder):
Initial patch for adding infinite scrolling to Themes screen (Currently
Installed Only).
This is a ''work in progress''.
Enabled fetch-list for class-wp-themes-list-table.
Ported wp-includes/js/wplink.dev.js, which handles infinite scrolling for
internal linking to wp-themes.dev.js for use in the Themes page.
Some things that are weird/known quirks:
* There aren't any UI elements (i.e. spinners) to show that we're loading,
but there are hooks in place to do this once the elements exist.
* Labels are not modified based on "how many" we've loaded or how many are
left. This means the paging always looks the same.
* As soon as you've loaded all themes, it just doesn't load anymore. This
is easy to hook into whatever we want to do when we're out of items.
* For now at least, most of parseQuery was copied from tb_parseQuery
(thickbox.js) into our JS for now, since making the change to thickbox.js
would change how the thickbox handles query strings (whether more correct
or not).
* The parsing of the features[] query array is a bit hackish, in part due
to how themes varies on how it includes it in the query string.
Suggestions for better methods of parsing are welcome.
* Yes, there is still some debug code, because it's a work in progress.
This will be removed before final, of course, is marked FIXME, and should
be obvious.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/19815#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list