The List 'Class attribute' is incredibly useful for specifying grids behaviors on Rows that use Div elements. Unfortunately the Class attribute gets completely ignored when the List type is set to None. The result is needlessly confusing and unintuitive.

This patch simply provides a basic Div option. See Screenshot

CommentFileSizeAuthor
semanticviews-div-option.patch792 byteszach harkey

Comments

leovw’s picture

leovw commented
Issue summary: View changes

This is a minor but really useful addition (especially so when working with themes based upon a framework such as bootstrap). Any chance this can be committed to both the Drupal 6 and Drupal 7 versions of Semantic views? It saves a fair bit of time (for me anyway).

Scyther’s picture

Scyther commented
Version: 6.x-1.x-dev » 8.x-2.x-dev

Is this needed for newer versions of semantic views?

leovw’s picture

leovw commented

I myself would say yes it is useful for D8 and should be an option. Predominantly using bootstrap very rarely do I want lists but very often I want divs!

Thanks

  • Scyther committed 389fb07 on 8.x-2.x authored by Zach Harkey 
    Issue #985432 by Zach Harkey, Scyther: Patch to Add 'Div' option for...
Scyther’s picture

Scyther commented
Status: Active » Fixed

I just needed this option today, so now I know why you needed it. Just added it to the dev version for D8. Will add it for D7 as well.

  • Scyther committed 99be3f6 on 7.x-1.x authored by Zach Harkey 
    Issue #985432 by Zach Harkey, leovw: Patch to Add 'Div' option for List...
leovw’s picture

leovw commented

Thanks!

Status: Fixed » Closed (fixed)

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

lennart’s picture

lennart commented
Version: 8.x-2.x-dev » 7.x-1.0

This very useful and simple option is not in the latest stable release for D7.

summit’s picture

summit commented

Hi, It is in 7.1 as 'div' => t('Division

') greetings, Martijn