Hello there,

I have tested this with a couple of admin themes but it's the same in both. BY default when enabling semantic views there is an option to add a row class attribute in the settings tab, then you can add the first and last row classes. The Row Class Attribute wasn't there by default like the upcomin screenshot.
Before Patch

I investigated it and found that it's there but not shown because a CSS bug with the FIRST/Last fieldset just after. Then I added a clear: both style to the mentioned fieldset so it appears like so:
After Patch

Thanks in Advance!

CommentFileSizeAuthor
#4 semanticviews-clearfix-1888258-4.patch2.44 KBTschet
sv.patch368 bytesibrahim.mohamed
Home Page (Content)2.png29.82 KBibrahim.mohamed
Home Page (Content).png26.43 KBibrahim.mohamed
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuaelFr’s picture

DuaelFr
he/him
Primary language French
Location Montpellier, France
CreditAttribution: DuaelFr commented
Status: Active » Needs review
smptebars1’s picture

smptebars1 CreditAttribution: smptebars1 commented

Just wanted to add. for those who may have difficulty understanding which file to add the code to. It is semanticviews_plugin_style_default.inc
Thanks for the fix.

joep.hendrix’s picture

joep.hendrix CreditAttribution: joep.hendrix commented
Status: Needs review » Reviewed & tested by the community

works
please commit

Tschet’s picture

Tschet CreditAttribution: Tschet commented
Issue summary: View changes
FileSize
semanticviews-clearfix-1888258-4.patch2.44 KB

The answer is simpler than it looks. Semantic Views 7.x is using the Drupal 6 "clear-block" instead of "clearfix". This patch corrects the class names.

Scyther’s picture

Scyther CreditAttribution: Scyther commented
Status: Reviewed & tested by the community » Needs work

Works, but the 'clearfix' should be wrappad in a array() like this array('clearfix')

  • Scyther committed 634f549 on 7.x-1.x 
    Issue #1888258: Change from clear-block to clearfix CSS class and wrapp...
Scyther’s picture

Scyther CreditAttribution: Scyther commented
Status: Needs work » Fixed

Fixed and commited to dev for 7.x.

Status: Fixed » Closed (fixed)

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