As a themer, it's maddening to find a display of related content not wrapped in a div which can easily be themed. In the case of the locations.tpl.php, there is no wrapping div surrounding both the <h3> title and the location display itself. It results in something pretty themer-unfriendly (for example, imagine wanting to place the location display somewhere, such as in a visual box or float the whole thing....)

The attached patch adds a wrapper div to bring both title and display together so it's easier to theme.

Comments

podarok’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev

all feature requests should be rolled against latest 7.x-3.x-dev and after commit can be backported to 6.x branch

Status: Needs review » Needs work

The last submitted patch, location-tpl-wrap.patch, failed testing.

mrtoner’s picture

Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new818 bytes

Agreed, wrapping in a div would make for easier theming.

  • podarok committed 2a19305 on 7.x-3.x authored by mrtoner
    Issue #1038164 by mrtoner, Rob_Feature: Added Wrap entire location...
podarok’s picture

Status: Needs review » Fixed

commited
Thanks!

Status: Fixed » Closed (fixed)

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