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

Comments

duaelfr’s picture

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

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.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community

works
please commit

Tschet’s picture

Tschet commented
Issue summary: View changes
StatusFileSize
new 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 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 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.