Closed mattheu closed 7 years ago
Found a bug in the pattern library that needs merging before launch. PR here: https://github.com/humanmade/hm-pattern-library/pull/93
Fixed. I'll update the version of the pattern library in use by this theme.
@mattheu I think I've got one or two more fixes in the pattern library first, sorry.
If there's docs on how to update this version, I could do it when I'm done? I noticed it's the same branch as gh-pages
? Is it just a manual push to that branch?
OK cool - I've not done it yet :)
You need to do a release on the hm-pattern-library. To do this you compile the project locally, and update the gh-pages branch with the contents of the dist directory. Its a little complex - maybe best I'll do it. I've been meaning to write a script to do this for you!
@mattheu Done browser testing now, there's just one more PR on the pattern library (https://github.com/humanmade/hm-pattern-library/pull/95) then we can release and update the refs here. I might try and get it working :)
Here's the PR for the browser fixes done within the handbook: https://github.com/humanmade/hm-handbook-theme/pull/79
Did you have anything specific in mind for the optimising? I can see the JS isn't minified, can fix that, but the CSS is. The SVGs are optimised through the pattern library. Anything else?
JS - yeah I'm pretty sure we can just tell webpack to build minified JS.
Anything else - nothing specific really, just take a look over see if there are any glaring issues.
I'd say this is all done now :)
Catch all ticket for all those small things we should do before making this thing public.