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
FileSize
392 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.