#9973 closed theme (live)
THEME: Coraline - 1.3
| Reported by: | automattic | Owned by: | kwight |
|---|---|---|---|
| Priority: | major | Keywords: | theme-coraline |
| Cc: | themes@… |
Description
Coraline - 1.3
A squeaky-clean theme featuring a custom menu, header, background, and layout. Coraline supports 7 widget areas (up to 3 in the sidebar, four in the footer) and featured images (thumbnails for gallery posts and custom header images for posts and pages). It includes styles for print and the Visual Editor, special styles for posts in "Asides" and "Gallery" categories, and has an optional full-width page template that removes the sidebar.
Theme URL - http://wordpress.org/extend/themes/coraline/
Author URL - http://automattic.com/
SVN - http://themes.svn.wordpress.org/coraline/1.3
ZIP - http://wordpress.org/extend/themes/download/coraline.1.3.zip?nostats=1
Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/coraline/1.2&new_path=/coraline/1.3
All previous tickets for this theme: http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-coraline&order=id
Change History (6)
comment:1 Changed 7 months ago by lancewillett
comment:2 Changed 7 months ago by kwight
- Owner set to kwight
- Status changed from new to assigned
comment:3 follow-up: ↓ 4 Changed 7 months ago by kwight
Why is the $locale bit removed from functions.php? Doesn't that break l10n?..
comment:4 in reply to: ↑ 3 Changed 7 months ago by lancewillett
Replying to kwight:
Why is the $locale bit removed from functions.php? Doesn't that break l10n?..
See http://core.trac.wordpress.org/ticket/20448:
a theme just doesn't need a $locale.php file — indeed, core almost never needs one either anymore. I suggest we remove the reference, something I had already done for Twenty Twelve when it was in trunk.
comment:5 follow-up: ↓ 6 Changed 7 months ago by kwight
- Resolution set to approved
- Status changed from assigned to closed
Kind of an odd UX that the Background Color is postMessage while Header Text Color isn't; it's a core thing, just hadn't noticed it before.
APPROVED. This is a partial and diff review.
comment:6 in reply to: ↑ 5 Changed 7 months ago by lancewillett
Replying to kwight:
Kind of an odd UX that the Background Color is postMessage while Header Text Color isn't; it's a core thing, just hadn't noticed it before.
Agreed. :)
Noting a few things like changing background to use add_theme_support() will be in the next version, didn't make it into this which is mostly bug fixes.