Getting an error message in the Drupal log reports from Sasson related to a phamlp file:

SassStringException: Value must be a unitless number:sites/all/themes/sasson/phamlp/sass/extensions/compass/frameworks/960/stylesheets/960/_grid.sass::7 Source: $fluid-gutter: $ninesixty-gutter-width / $ninesixty-grid-width * 100% in SassString->op_times() (line 71 of /sites/all/themes/sasson/phamlp/sass/script/literals/SassString.php).

Additionally there is a related Drupal "page not found" error in the log reports citing this URL:

sites/all/themes/sasson/phamlp/sass/extensions/compass/frameworks/960/stylesheets/960/_grid.sass::7

Looks like this library is removed for 7.x-3.x dev branch, so this is only an issue for the 7.x-2.x branch.

Attached diff shows a potential fix, removing the % sign when calculating a percentage as documented here:

http://sass-lang.com/docs/yardoc/Sass/Script/Functions.html#percentage-i...

CommentFileSizeAuthor
Sasson100.diff151 byteszirafa
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zirafa’s picture

zirafa CreditAttribution: zirafa commented
Issue summary: View changes

formatting

tsi’s picture

tsi CreditAttribution: tsi commented

Yeah, Sasson v3.x uses a completely different layout system, and a different compiler too, so it won't be an issue.
I don't understand the fix, what you suggest will return a different result then intended:
Say $ninesixty-gutter-width: 10px and $ninesixty-grid-width: 960px -
10px / 960px * 100% != 10px / 960px * 100

What I would try, according to the Sass docs you link to is this:
$fluid-gutter: percentage($ninesixty-gutter-width / $ninesixty-grid-width);

zirafa’s picture

zirafa CreditAttribution: zirafa commented

Yes you are right the correct syntax appears to be:

$fluid-gutter: percentage($ninesixty-gutter-width / $ninesixty-grid-width)
tsi’s picture

tsi CreditAttribution: tsi commented

Did you test it? does it solve your problem?

zirafa’s picture

zirafa CreditAttribution: zirafa commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

editing url