Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This patch upgrades Zurb Foundation to 6.2, in addition to bringing @bcwald's Gulp work back to the base theme.
I haven't touched the templates yet, so we'll need to make sure that features like the Top Bar still work before committing this.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-2702271-3-4.txt | 520 bytes | samuel.mortenson |
#4 | zurb-foundation-update-6.2-2702271-4.patch | 1.65 MB | samuel.mortenson |
#3 | zurb-foundation-update-6.2-2702271-3.patch | 1.65 MB | HongPong |
#3 | interdiff-2702271-2-3.txt | 1.46 KB | HongPong |
#2 | interdiff-2702271-1-2.txt | 3.92 KB | samuel.mortenson |
Comments
Comment #2
samuel.mortensonUpdated patch, which fixes SASS compilation errors in the subtheme as well as fixing off-canvas (which currently does not work. Top bar, sticky, nested menus, and the mobile menu icon all seem to work.
Please review so we can get this in!
Comment #3
HongPong CreditAttribution: HongPong at kor group commentedA couple notices I got at `bower install` worth looking at resolving:
Drupal 8.0.6 is distributed with jQuery 2.1.4. I think it should be okay if 2.2.3 is running instead.
I changed the dependency versions to
which unifies the jquery requirement to 2.2.3. Also changed the bower "name" to avoid complaints from system. I also added a little more helpful info to the gulpfile to clue the developer in to what it's doing.
Also I checked out all the settings with topbar and it seems like they come through OK as long as you make sure to rebuild the cache, it can stick to the top of the browser window correctly including when the administrative menu is stuck there, it attaches right under it correctly for admins.
Comment #4
samuel.mortensonThanks for updating the version - to prevent this in the future we should ignore jQuery as a dependency (which we already did in the STARTER theme). This patch updates the STARTER theme's dependencies to match the base theme as well as ignoring jQuery in the base theme.
Comment #5
HongPong CreditAttribution: HongPong at kor group commentedI got the following notices regarding various packages, however these are not due to problems in the theme itself.
the lodash and graceful-fs warning is from here:
The notice on npmconf derives from gulp-sass@2.3.1 and node-sass@3.5.3.
Anyhow I think we can call this RTBC, as it is a big patch and npm packages are not immediately a blocking problem. I will try to have a better README.txt patch tomorrow that reflects the new workflow, as seen in the STARTERKIT readme. Gotta love those sprawling npm dependency trees. I'm sure their noise will alarm Drupal devs forever now.
Comment #7
samuel.mortensonDone!
I accidentally pushed the wrong commit message and had to do a merge as Drupal.org won't let you force push after amending a commit message, but you should have the commit credit now.
Comment #8
HongPong CreditAttribution: HongPong at kor group commentedSweet thanks for throwing that in there. Tag me on anything that blocks an alpha release for the next week or so I can try to knock out. Cheers