Skip to content

fix: Tweak badge height as per design mocks#757

Merged
engfragui merged 2 commits intomainfrom
francesca/tweak-badge
Feb 21, 2023
Merged

fix: Tweak badge height as per design mocks#757
engfragui merged 2 commits intomainfrom
francesca/tweak-badge

Conversation

@engfragui
Copy link
Copy Markdown
Contributor

@engfragui engfragui commented Feb 20, 2023

Short description

Minor follow-up to #755, after testing the new version in todoist-web. Looks like this:

Screenshot 2023-02-20 at 16 42 11

which looks too tall.

However, what we know is that:

  • font size should be (and is) 10px
  • height of the badge should be 15px (as per mocks)

Therefore the padding should be 2.5px at the top and 2.5px at the bottom. I had 3px and 3px before this change. Apologies, I hadn't checked the mock carefully enough apparently.

PR Checklist

  • Added tests for bugs / new features
  • Updated docs (storybooks, readme)
  • Executed npm run validate and made sure no errors / warnings were shown
  • Described changes in CHANGELOG.md
  • Bumped version in package.json and package-lock.json (npm --no-git-tag-version version <major|minor|patch>) ref
  • Updated all static build artifacts (npm run build-all)
  • Reviewed and approved Chromatic visual regression tests in CI

Versioning

Patch bump.

@engfragui
Copy link
Copy Markdown
Contributor Author

engfragui commented Feb 20, 2023

@gnapse I'm running into an issue in the "Publish to Chromatic" build:

Error: error:0308010C:digital envelope routines::unsupported

Based on "the internet", it seems like this is due to fact that that build is running on Node v18, even though I don't think we're actually intentionally supporting it in Reactist.

Have you encountered this before? I feel like a possible solution could be to ensure that we're using node v16 in the chromatic configs 👀 Looking now.


Edit: Errr actually not sure this is popping up now. I don't think we updated any dependencies recently... 🤔


Edit: It actually started failing even before this PR, when I merged my previous PR. See: https://github.com/Doist/reactist/actions/runs/4224761740/jobs/7336138735

The PR that caused that build failure was #755

It's possible that the output is confusing and this is just an issue with my storybook stories since the output also says:

    → Running command: npm run --silent build:storybook -- --output-dir /tmp/chromatic--1693-GA2nfUaC9H7x --webpack-stats-json /tmp/chromatic--1693-GA2nfUaC9H7x
    → [*                   ]
The CLI tried to run your build:storybook script, but the command failed. This indicates a problem with your Storybook. Here's what to do:

Edit: Running

npm run build:storybook

locally works fine (no errors). 🤷‍♀️

@engfragui engfragui changed the title fix: Exact badge height as per design mocks fix: Tweak badge height as per design mocks Feb 20, 2023
@engfragui engfragui force-pushed the francesca/tweak-badge branch from eb622b1 to 7eff956 Compare February 21, 2023 10:45
@engfragui
Copy link
Copy Markdown
Contributor Author

Fix for the Chromatic build is in #758

@engfragui engfragui force-pushed the francesca/tweak-badge branch from 35d1606 to e5359cd Compare February 21, 2023 12:35
@engfragui engfragui merged commit 8e73065 into main Feb 21, 2023
@engfragui engfragui deleted the francesca/tweak-badge branch February 21, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant