ibm-products icon indicating copy to clipboard operation
ibm-products copied to clipboard

cspell fixes

Open davidmenendez opened this issue 1 year ago • 2 comments

CSpell: Files checked: 1241, Issues found: 53 in 19 files

these are the results after fixing the broken spellcheck command.

most of these appear to be under coachmark so i'll tag @jlongshore & @paul-balchin-ibm to take a look

davidmenendez avatar Feb 12 '24 15:02 davidmenendez

@elycheea to do a first review of what’s being flagged.

elycheea avatar Feb 15 '24 17:02 elycheea

Going to open a PR for the quick fixes.

For the remaining issues errors, run yarn spellcheck --quiet to see remaining issues. These occur both in the ibm-products-styles and ibm-products packages.

src/components/Coachmark/_coachmark-tagline.scss:39:27 - Unknown word (ctacol)
src/components/Coachmark/_coachmark-tagline.scss:39:41 - Unknown word (closebtncol)
src/components/Coachmark/_coachmark-tagline.scss:89:29 - Unknown word (iconcol)
src/components/Coachmark/_coachmark-tagline.scss:89:51 - Unknown word (bodycol)

src/components/Guidebanner/_guidebanner.scss:251:12 - Unknown word (Leftgutter)
src/components/InlineTip/_inline-tip.scss:232:12 - Unknown word (Leftgutter)

src/components/Decorator/Decorator.mdx:12:38 - Unknown word (setlabeltitle)
src/components/DecoratorBase/DecoratorBase.mdx:15:38 - Unknown word (setlabeltitle)
src/components/DecoratorDualButton/DecoratorDualButton.mdx:15:38 - Unknown word (setlabeltitle)
src/components/DecoratorLink/DecoratorLink.mdx:15:38 - Unknown word (setlabeltitle)
src/components/DecoratorSingleButton/DecoratorSingleButton.mdx:15:38 - Unknown word (setlabeltitle)

src/components/Nav/Nav.js:104:69 - Unknown word (Pageload)
src/components/Nav/Nav.test.js:116:25 - Unknown word (Pageload)
src/components/Nav/NavList.js:24:15 - Unknown word (Pageload)
src/components/Nav/NavList.js:41:19 - Unknown word (Pageload)
src/components/Nav/NavList.js:41:51 - Unknown word (Pageload)
src/components/Nav/NavList.js:50:50 - Unknown word (Pageload)
src/components/Nav/NavList.js:60:57 - Unknown word (Pageload)
  • Coachmark SCSS *-col can probably hyphenate? 🤔
  • leftgutter should probably also be hyphenated?
  • setLabelTitle is used for anchor link in MDX so may need to look into that one
  • Nav needs either to be added as an exception pageload or a prop in NavList should be renamed isExpandedOnPageload to isExpandedOnPageLoad. Latter is more work but probably what we would have recommended if Cspell had been running properly.

elycheea avatar Apr 04 '24 20:04 elycheea