[wp-trac] [WordPress Trac] #24184: Twenty Thirteen: remove fixed navbar
WordPress Trac
noreply at wordpress.org
Fri May 3 04:52:50 UTC 2013
#24184: Twenty Thirteen: remove fixed navbar
---------------------------+------------------
Reporter: lancewillett | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.6
Component: Bundled Theme | Version:
Severity: normal | Resolution:
Keywords: has-patch |
---------------------------+------------------
Comment (by nacin):
Good points here. I'm still a pretty strong no.
This is not just the newest default theme for WordPress. This is a theme
that hundreds of thousands of people are going to open up and tear apart.
If the original developers involved in putting Kubrick together knew most
of us would have learned WordPress theming through it, do you think they
would have done X, Y, and Z differently?
A default theme should be elegant. Not just in design, but in code. It
needs to be simple. Too clever is cool on occasion, and is often just fine
when it is buried in some internals. Too clever is not cool when people
'''who are not programmers''' are going to be dissecting this theme and
those who are programmers using it as a '''baseline''' for the next 12-24
months.
The standard we hold a default theme should be unique. It is most
certainly more constraining — by its necessity as a standard-bearer and as
a learning tool. It's because it sets a particular bar: "''this'' is how
you code a theme." And yet, despite those constraints, we have a pretty
great, bold design here.
This theme makes more than enough of a statement when it comes to design.
Does it ''really'', ''honestly'' need this too? Twenty Thirteen is nearly
2,000 lines of PHP. That is a good chunk of code. (Also 5,000 lines of
CSS.) We're now talking specifically about 126 lines of not uncomplicated
JavaScript, and for what? This is not amazing, nor is it must-have. This
is scroll-to-top, some nifty fixed positioning, and what really just looks
like a lot of code cruft and an over-the-top bell/whistle we couldn't part
with because reasons.
(Side note: if we bundled vanilla masonry (http://vanilla-
masonry.desandro.com/), we could get rid of the jQuery dependency all
together.)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/24184#comment:17>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list