[wp-trac] [WordPress Trac] #21785: Add header image uploads with cropping to the customizer

WordPress Trac noreply at wordpress.org
Wed Feb 26 19:03:26 UTC 2014


#21785: Add header image uploads with cropping to the customizer
----------------------------+-----------------------------
 Reporter:  nacin           |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  Future Release
Component:  Appearance      |     Version:  3.4
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:  javascript
----------------------------+-----------------------------

Comment (by mcsf):

 Updated patches
 [https://core.trac.wordpress.org/attachment/ticket/21785/21785-media-
 manager.2.diff 21785-media-manager.2.diff],
 [https://core.trac.wordpress.org/attachment/ticket/21785/21785-customize-
 header.2.diff 21785-customize-header.2.diff] and
 [https://core.trac.wordpress.org/attachment/ticket/21785/21785-tests.2.diff
 21785-tests.2.diff] address feedback from @nacin and @ocean90, namely:

 * Replaced image (the "close" PNG) with Dashicon
 * Ditched slimScroll

 * AJAX: Added missing capabilities check
 * AJAX: action adds alternatively use `$this` and `__CLASS__`

 * L10n: Added missing l10n for "No image set" placeholder
 * CSS: Removed wpcom-ism (`body:not(.blue)`)

 * JS: Moved Header views and models from `wp-admin` to `wp-includes`
 * JS: Ditched the template in `wp.media.view.Cropper`

 * PHP: In `WP_Customize_Header_Image_Control::render_content`: braces
 around if and else blocks.
 * PHP: Use `wp_die` over `die`

 * Tests: have test classes extend `WP_UnitTestCase`
 * Tests: Move JS test fixtures to `index.html`
 * Check that JSHint rules are followed

 * (not from feedback) Cleanup: Removed leftover parent attachment logic
 from previous feature

 For reference, we're doing development at [https://github.com/ehg
 /wordpress-develop/ github:ehg/wordpress-develop]; use
 `/commits/dev?author=AUTHOR` with `AUTHOR` in `(ehg|mcsf)` for granular
 commits.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/21785#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list