Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
@see title.
What about this? There is a weird bug with sass-globbing that prevents it from properly working when you try to import with wildcards from a sub-folder of /sass. At least it did not work for me in the ohm-hero layout .scss files.
Also, having the layouts folder in /sass directly next to the partials folders is kinda werid... It's a pitty that we can't have multiple Sass source folders :/.
Comment | File | Size | Author |
---|---|---|---|
#9 | omega.code_.2089383-9.patch | 13.47 KB | fubhy |
#8 | omega.code_.2089383-8.patch | 13.47 KB | fubhy |
#1 | omega.code_.2089383-1.patch | 13.5 KB | fubhy |
Comments
Comment #1
fubhy CreditAttribution: fubhy commentedOh... Forgot to upload the patch!
Comment #2
fubhy CreditAttribution: fubhy commentedComment #3
fubhy CreditAttribution: fubhy commentedNote: I also moved some of the form stuff from /base to /components (e.g. the password related one, etc.) as we previously agreed that "base" should be considered pure HTML elements only (with some, rare exceptions).
Comment #4
brst t CreditAttribution: brst t commented20131102 - removed by author
Comment #5
fubhy CreditAttribution: fubhy commentedWe don't want to have multiple config.rb files. Everything should compile with one compiler running. One should not have to go to multiple places in the theme to run "compass watch". Hence, we moved all .scss files to the global /sass folder so a single compiler run watches all files in the theme. The problem with that is that we can't place the .scss files of our layouts where they actually belong (the folder of the corresponding layout).
This is a limitation of Compass. One can not have multiple source/destination folders. With plain sass that is possible
sass watch sass:css layouts:css/layouts --compass
That command would compile everything from /sass into /css and everything from /layouts into css/layouts.
But that does not work with "compass watch" (or the config.rb rather).
Comment #6
brst t CreditAttribution: brst t commented20131102 - removed by author
Comment #7
msmithcti CreditAttribution: msmithcti commentedReally like the restructuring, that does make thing simpler, and prevents us from having two concepts of layouts with in the sass/ directory.
On the base vs. component grey area, generally base styles should be just HTML elements and this is how I explain base styles to people. In some cases (especially in Drupal) this isn't fully attainable, so we shouldn't be too strict in always applying this rule. The more complex groups of elements (like the password) should definitely be a component. My only problem with #1 that we are now splitting up the styling for fieldsets and form items in two different places, which IMO adds confusion.
Comment #8
fubhy CreditAttribution: fubhy commentedOkay, granted... form items and fieldsets can stay there...
How do you like this then? The thing with compass not supporting multiple sass_dir and css_dir declarations is really annoying, meh.
Comment #9
fubhy CreditAttribution: fubhy commentedQuickly fixing that new-line problem.
Comment #10
fubhy CreditAttribution: fubhy commentedOkay, imho this is good enough to get committed as a first step. We can still tinker with the layouts later. I also changed the extended starterkit accordingly. If there is anything more you would like to improve please open a second step issue and post it here.
Comment #11
brst t CreditAttribution: brst t commented20131102 - removed by author
Comment #12
fubhy CreditAttribution: fubhy commented.scss, not .css :P
Comment #13
brst t CreditAttribution: brst t commenteddhrr.
Fixed