Closed jmcc0nn3ll closed 9 months ago
Note that the UI project is configured to use ES5, which means no let
, const
, multiline strings, or stabby functions. We can look to migrate the build to allowing ES6, but it's just not set up that way at the moment (because it inherits from the default UI, which has yet to make that transition).
We may want to consider splitting the CSS (and maybe even the JS) for home area off from that used by the rest of the site. For example, home.css loaded after site.css. However, that could be a premature optimization. I just wanted to mention that it is possible.
It would be good to add a preview page in the UI project so that this can be tested without building the whole site.
Can you send a pointer to set up a preview?
I have updated the PR to make this a more seamless integration. Here are the key changes:
The appearance of the cards could still use work. I did not really fiddle with the design.
In the future, can you please submit PRs from a fork? Making branches in the upstream makes it messy.
Sure, I'll do that, though not sure what I'll do beyond content for the time being.
thanks!
Adds support for blog cards for the front page. Colors are temporary but seem flexible enough for what we want to do overall.