The code and comment don't match, the code defines the span as 8 but the comment says it's 12.

.two-sidebars {
.l-content {
@include span-columns(8, 16); // Span 12 out of 16 columns.
@include push(4, 16); // Push element by adding 4 out of 16 columns of left margin.
}

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BiigNiick’s picture

Status: Active » Needs review
FileSize
602 bytes

ok, i think this fixed it.

- nick

Status: Needs review » Needs work

The last submitted patch, omega-fix_comment_ simple.layout.scss-2070139-1.patch, failed testing.

BiigNiick’s picture

oops, had a space in the file name.

fixing file name for test bot

- nick

BiigNiick’s picture

Status: Needs work » Needs review
BiigNiick’s picture

Title: Error in code line 112 of simple.layout.scss » Error in code line 105 and 112 of simple.layout.scss
FileSize
915 bytes

also an error in line 105. new patch

rocketeerbkw’s picture

Version: 7.x-4.0-beta7 » 7.x-4.x-dev
Status: Needs review » Reviewed & tested by the community

looks good to me, applies cleanly to HEAD

fubhy’s picture

Status: Reviewed & tested by the community » Fixed

Good catch, thanks.

Status: Fixed » Closed (fixed)

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