By Developers, For Developers
PDF Pg | Paper Pg | Type | Description | Fixed on | Comments |
---|---|---|---|---|---|
12 | TYPO | “of is” in the following sentence: The grand aim of is SCSS to keep the look of CSS while introducing the units of Sass. | 2011-09-05 | Thank you! \n–michael | |
12 | TYPO | It appears that a period should follow CSS in the following sentence(s). It stands for Sassy CSS If you’re familiar with CSS, it’s pretty easy to read. | 2011-09-05 | Oops. Thanks for spotting that! | |
22 | ERROR | Under #9 “Creating Themes with Advanced Colors”, in page# 22, the second half of the page has code that looks like this: #page { I think this should be #page { This style occurs twice. Also, the same page has the following: grayscale(#336699); The “->” is misleading (thought what follows after that symbol was meant to be the final css). I think both are equivalent to get the same output, and so can be changed to something like: grayscale(#336699); | 2011-09-05 | Thanks for spotting this! | |
36 | SUGGEST | I thought we could talk about two more things while talking about mixins: 1. the ability to accept default parameter/descriptor values I have illustrated both in the below snippet: =blue_text($size : 20px) .product_title .product_summary Hope this would help. | 2011-09-06 | I forgot we hadn't added those bits in. Thanks for suggesting this! | |
38 | SUGGEST | “We’ve worked really hard on the Sass development team to make sure that the error messages make as much sense as possible.” Shouldn’t this be: “We’ve worked really hard to make sure that the error messages make as much sense as possible.” “worked really hard on the Sass development team” — doesn’t it mean working hard ON the team rather than the intended meaning of “The team worked hard”. Or is my [Queen’s-Indian] English not up to the mark? :) | 2011-09-05 | I agree that wasn't that clear beforehand. Thanks! | |
42 | SUGGEST | Might be better to introduce variable/parameter interpolation? The associated code in page# 43 has this: filter: alpha(opacity=#{$opacity}) Are we assuming that readers will know Ruby and so would understand “#{$opacity}”? I have read other sources and know that interpolation is available in Sass (similar to in Ruby strings). But not everyone would know, right? | 2011-09-06 | We introduce interpolation in the next task... so I think this is OK, maybe? | |
46 | SUGGEST | Can you provide some [1 or 2 line] guidance in the book on when @each should be preferred and when a mixin should be preferred? For instance, I could achieve the same using this: =member_info($member) +member_info(thom) Yes, there is copy-paste, but the advantage is I can move the mixin to a separate file, isn’t it? | 2011-09-07 | I think the fact that there's a lot of copy-paste is why @each is preferred over a mixin. Plus, it's more common to use @each in SassScripts - especially when you have potentially a large number of variables. | |
48 | TYPO | There is a line that reads (in my pdf on a Ubuntu 10.10 box using Document Viewer v2.32.0): “For example, 20 10 would evaluate to true.” I think an operator is missing.. may be “>”? | 2011-09-05 | Bingo! We'd used the wrong xml entity. Whoops | |
51 | ERROR | In the code for section 21, for the last two snippets (portrait and landscape), we are missing indentation for the last line: @media screen and (max-width: 320px) @media screen and (min-width: 321px) and (max-width: 480px) I know this must be a typo, but since it affects the final output (without indentation, the @media is stripped out from the generated css), I am logging this as a Tech Error. | 2011-09-05 | Yup, that is a typo. But one with pretty important consequences. \nThanks! | |
65 | SUGGEST | I am on Compass 0.11.5 (Antares). In the horizontal list, when I don’t pass any params, I am getting the padding as 4px, as against the 10px in the book. What gives? @import compass/typography/lists/horizontal-list -> ul.horiz li { | 2011-09-05 | Sorry - I don't really know what happened there. It should be 4px. Thanks! \n–michael | |
66 | SUGGEST | This line can be improved? “use” is used twice and the sentence somehow doesn’t gel. “Within the HTML we have to use to describe the sticky footer, there are three pre-defined selectors that we use in this mixin —root, root_footer and footer.” | 2011-09-05 | Hopefully I've made this a bit clearer. | |
76 | SUGGEST | I am relatively new to CSS/Sass/Compass/Sprites. I have been having a tough time getting the Sprites thing to work. It would be great if there is a bit more detail available as to how to set up the project (e.g. there should be an images folder, inside which the icon folder should go, and so on). Also, if there are any other pre-requisites, it would be useful to have them listed out as well. I am on: My code in the sass file is:
I tried setting up the images/icon/*.png, then tried with the commented line above, and since it didn’t work, switched to trying out “compass sprite” command, and tried with the above version. [That command didn’t create anything other than the file sass/sprites/_icon.scss]. The error messages I get are (at different times, when I tried the aforementioned things): error sass/spritest.sass (Line 18: Invalid CSS after “…$icon-clean-up,”: expected function argument, was “);”) NoMethodError on line [“24”] of /home/proto/.rvm/gems/ruby-1.9.2-p180@rails3/gems/chunky_png-1.2.1/lib/chunky_png/chunk.rb: undefined method `unpack’ for nil:NilClass Since it is quite possible that others too might face this issue, would be great if you can provide a bit more details (and gotchas). Thanks! | Where are your files located? \n \nSomeone else had a similar error... \nhttp://groups.google.com/group/compass-users/browse_thread/thread/8805c8187e794610/0cc0a34513de387f?lnk=raot \n \nWhere are your images stored exactly? | ||
12 | TYPO | The second sentence in the “SCSS: A More CSS-like Way to Write Sass” section should read “The grand aim of SCSS is to keep the look of CSS while introducing the units of Sass.” instead of “The grand aim of is SCSS to keep the look of CSS while introducing the units of Sass.” | 2011-09-05 | Cheers, I've fixed this :) | |
65 | TYPO | Last sentence, first paragraph reads: “If you want to remove only one bullet point, just @includeno-bullet in the class.” Missing the space between “@include” and “no-bullet”. | |||
12 | ERROR | " Just place your comment between /* and */. These comments can be on multiple lines. One will only show up in the Sass document, and the other will be incorporated into the CSS that’s compiled." This doesn’t seem to be true, if you are compiling with Scout to compressed output style, no matter if production or deployment has been chosen as environment. | |||
7 | ERROR | Convert a CSS file to a Sass file Note the hyphen is required in lieu of the space | |||
88 | TYPO | In the second paragraph, | |||
106 | TYPO | First paragraph: |product|variable is missing a space. - RR | |||
21 | ERROR | The SCSS example has one too many } at the end. so : should be: | |||
37 | TYPO | In the section talking about @extend, the code listed reads .checkout_button { However, that code, as it stands, produces nigh-unreadable code, and anyone that’s blue colorblind can just forget it. I believe the effect he was looking for is better expressed .checkout_button { which should render the proper effect. |