Opened 3 months ago
Last modified 3 months ago
#23504 new task (blessed)
Twenty Thirteen
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Bundled Theme | Version: | trunk |
| Severity: | normal | Keywords: | |
| Cc: | travis@…, lachlan@…, bronson@…, jeremy+wp@…, jartes, Frank, Klein, tracy@…, xoodrew@…, mdhansen@… |
Description
The Twenty Thirteen team has handed the theme over, and we're ready to land it in core and start smoothing out the rough edges. Open new tickets with a component of "Bundled Theme" to track this, and try to attend the Twenty Thirteen office hours on Tuesdays and Thursdays at 1700 UTC.
Change History (21)
comment:1
markjaquith — 3 months ago
Lack of clearing for the sidebar/footer: http://cl.ly/image/0g1R1F3y0Z3V
comment:4
travisnorthcutt — 3 months ago
- Cc travis@… added
@nacin trying to reproduce that. What steps led to that?
- Cc jartes added
@travisnorthcutt It happens when the height of the main content is lower than the sidebar:
http://joanartes.com/wp-content/uploads/twentythirteen.png
http://joanartes.com/wp-content/uploads/twentythirteen2.png
comment:9
Frank Klein — 3 months ago
- Cc Frank, Klein added
Besides the clearing issue, maybe the footer could stick to the bottom of the viewport if the content is shorter?
comment:10
taupecat — 3 months ago
- Cc tracy@… added
comment:11
lancewillett — 3 months ago
Non-clearing footer is by design. This theme is intended to be used without a sidebar; if one is enabled, the footer will be just another "band of color" below the content.
comment:12
follow-ups:
↓ 14
↓ 17
markoheijnen — 3 months ago
It's fine that the theme is not intended to have a sidebar but when there is one it should be dealt better. I guess almost everyone would say it is broken how it is now.
comment:13
WraithKenny — 3 months ago
Any news on when genericons.com will be made not private? It's mentioned in the css as an info link, but it's unavailable to the public as of yet.
comment:14
in reply to:
↑ 12
lancewillett — 3 months ago
Replying to markoheijnen:
It's fine that the theme is not intended to have a sidebar but when there is one it should be dealt better. I guess almost everyone would say it is broken how it is now.
I thought so, too ... but after working with it for a week or two I've changed my mind. :) Let's give it some time.
comment:15
DrewAPicture — 3 months ago
- Cc xoodrew@… added
comment:16
in reply to:
↑ 2
SergeyBiryukov — 3 months ago
Replying to nacin:
Lack of clearing for the sidebar/footer: http://cl.ly/image/0g1R1F3y0Z3V
Also reported in #23557.
comment:17
in reply to:
↑ 12
MikeHansenMe — 3 months ago
Replying to markoheijnen:
It's fine that the theme is not intended to have a sidebar but when there is one it should be dealt better. I guess almost everyone would say it is broken how it is now.
I agree, it was the first thing I noticed in the theme. I think that is the general consensus as well. We can give it time but I think it is *broken*
comment:18
MikeHansenMe — 3 months ago
- Cc mdhansen@… added
comment:19
nacin — 3 months ago
Note that the vast majority of individuals will have a single widget sidebar configured (if only because that is what core has done for them since 3.0), which will automatically map to the first sidebar.
It's a "footer", it shouldn't be left dangling midway up the page like that. I somewhat understand the design sentiment but it looks broken and everyone will think it is broken and unfinished.
If the goal was for the footer sidebar to be first, then it should be registered first. Sidebar descriptions should make it clear that the footer widget area is primary, while the sidebar is optional, and that Twenty Eleven looks great single-column.
comment:20
lancewillett — 3 months ago
but it looks broken and everyone will think it is broken and unfinished
That leaves little room for the non-everyone people, doesn't it. :)
comment:21
lancewillett — 3 months ago
Discussed the sidebar and clearing footer today in office hours. Notes here: http://make.wordpress.org/core/2013/02/21/twenty-thirteen-project-update/

In 23452: