Hi,

I think footer tag should be at same level as header tag, out of main-container div, so we could apply subthemes with full width footer without rewriting template files.

Greets!

Files: 
CommentFileSizeAuthor
#1 1876300_footer_out_of_main_container_0.patch392 bytesmaxtorete
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

maxtorete’s picture

Status:Active» Needs review
StatusFileSize
new392 bytes
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Patch attached!

markcarver’s picture

Status:Needs review» Fixed

Agreed. Fixed in commit a26149d. FYI, the footer tag also has the .container class. To obtain full width across the screen, you'll probably have to remove it and set a wrapping div with it inside it.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.