gesso-wp icon indicating copy to clipboard operation
gesso-wp copied to clipboard

Removes design tokens, scripts, references, updates theme.json, etc.

Open cssgirl opened this issue 1 year ago • 7 comments

Testing AC:

  • [ ] Check that correct packages are installed and build/watch commands work as expected
  • [ ] Make sure all references to gesso functions and variables have been removed and replaced with the equivalent CSS variable generated from theme.json
  • [ ] Make sure updated patterns, templates work as expected
  • [ ] Review theme.json updates & additions
  • [ ] Check that the site editor options we want are available and the ones we are not ready to use yet are disabled/hidden

cssgirl avatar Apr 23 '24 13:04 cssgirl

@cssgirl Something I'm noticing - npm run rename doesn't seem to work. I don't think this was caused by your changes - it's not working in the base branch either. Seems like the dependencies aren't getting installed. Can you look into that?

Maybe we just need to add the dependencies to package.json manually?


const inquirer = require('inquirer');
const path = require('path');
const fsPromises = require('fs/promises');

jackmakesthings avatar May 08 '24 16:05 jackmakesthings

@cssgirl Thank you! Would you mind dropping some testing AC in here so we can stress test the changes? Just any editorial expectations that you think might have been effected that we can verify.

tropicandid avatar May 08 '24 19:05 tropicandid

@tropicandid @jackmakesthings - I've added some preliminary AC above!

I am updating the readme and have one small batch of changes (color variable names) - otherwise this should be ready to pull down and test!

cssgirl avatar May 10 '24 18:05 cssgirl

Will we update the gesso version number once this is merged?

jackmakesthings avatar May 15 '24 17:05 jackmakesthings

Last item I am working on - fixing the theme's rename functionality.

cssgirl avatar May 15 '24 17:05 cssgirl

@cssgirl Something I'm noticing - npm run rename doesn't seem to work.

@jackmakesthings Fixed! Looks like it never actually worked. Seems like it was a copy of the same rename functionality we have in the block library and some variable names & stuff and the inquirer package were overlooked. Should be working now!

cssgirl avatar May 16 '24 15:05 cssgirl

One last set of changes to be made:

  1. remove all CSS for blocks provided by the f1 block library (except for cards) and move those styles into the appropriate block folders in the f1 block library.
  2. Update references to images in kitchen sink pattern (and other patterns as needed) so no broken images.

cssgirl avatar May 16 '24 15:05 cssgirl

@cssgirl I am seeing two small things OOTB. I've attached screenshots.

  1. In the editor, now that we've removed the colors, when I select/highlight content, the text goes white and is illegible.
Screenshot 2024-05-20 at 3 01 24 PM
  1. The pattern for the Article displays the block recovery message by default.
Screenshot 2024-05-20 at 3 04 54 PM

tropicandid avatar May 20 '24 19:05 tropicandid