[wp-trac] [WordPress Trac] #25858: Integrate MP6 into core

WordPress Trac noreply at wordpress.org
Tue Nov 12 22:24:28 UTC 2013


#25858: Integrate MP6 into core
----------------------------+------------------
 Reporter:  dd32            |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  3.8
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |
----------------------------+------------------

Comment (by tollmanz):

 25858.6.diff​ is a first pass at the merge of the sticky menu component
 into core. In the merge process, I have updated a few things:

 * I added a debounce function to the resize and scroll events. It is
 important that this is throttled
 * Everything was cleaned up to pass the jshint rules
 * All repeated selectors were cached

 A few questions about this:

 1. Given the integration into core via add JS to an existing JS file and
 CSS to an existing CSS file, I removed the PHP related to this component
 that only served to load the assets. One piece of that PHP is that it did
 not load this JS/CSS on mobile other than iPad. I could not figure out why
 this was a requirement. Is this needed? Obviously, where possible,
 especially with a project like WordPress, browser sniffing is not
 desirable, so it would be nice to remove this altogether, but I do not
 have an idea as to the implications of it.
 1. I stole Underscore.js's debounce function. I did not want to include
 Underscore.js as it would make Underscore.js a dependency for all of WP
 admin. Any thoughts use of this function?

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25858#comment:19>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list