Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#23662 closed enhancement (wontfix)

Twenty Twelve: style.css with Table of Contents

Reported by: tomasm's profile TomasM Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Hello, I really like the 2013 initiative with Table of Contents, because it helps new users understand where goes what, especially developing child themes.

I added Table of Contents to Twenty Twelve theme and renamed CSS comments titles accordingly.

Attachments (2)

style with Table of Contents.css (36.0 KB) - added by TomasM 11 years ago.
23662.patch (8.4 KB) - added by ocean90 11 years ago.

Download all attachments as: .zip

Change History (11)

@ocean90
11 years ago

#1 @TomasM
11 years ago

Thanks, ocean90! ;)

#2 @DrewAPicture
11 years ago

  • Cc xoodrew@… added
  • Keywords has-patch added

+1 for 23662.patch

#3 @TomasM
11 years ago

@DrewAPicture I noticed, that for some reason "7.0 - Aside post format" was separated from other post formats in the section 8.0. I didn't want to take a freedom and mess with the structure, but perhaps it would be good to move that peace of code down under "8.0 - Post formats"?

Last edited 11 years ago by TomasM (previous) (diff)

#4 @SergeyBiryukov
11 years ago

  • Version changed from trunk to 3.5

#5 @lancewillett
11 years ago

  • Keywords dev-feedback removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Let's skip this, and put our efforts to newer themes instead. Thirteen and onward.

I don't think it's worth the effort for the older themes.

#6 @TomasM
11 years ago

  • Resolution wontfix deleted
  • Status changed from closed to reopened

I'm sorry but I disagree. There is nothing to fix, everything is done and patch is ready.

You can focus on new theme, but I'm focusing on a theme, that I really like, like many others. And this patch will improve understanding about this theme for new users.

Plus 2012 is much better starting point for normal website. 2013 is good only for a blog.

We spent our time for this and this adds value for the whole community.

#7 @lancewillett
11 years ago

  • Milestone set to Future Release

It's still a maintenance burden regardless of focus, and doesn't provide enough value to warrant the change right now. I'd be open to discussing this early in the 3.7 cycle, including refactoring all the stylesheets from default themes to add better organization and clean everything up to match WP CSS style guidelines.

#8 @TomasM
11 years ago

If you would add it now, it would be a precedent, something to work on later or extend. Otherwise it might be buried.

And "refactoring all the stylesheets" - that would be a real burden. This patch fixes 90% of problem without more work to do if it would be decided.

#9 @lancewillett
11 years ago

  • Keywords has-patch removed
  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.