cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

refactor(web): remove `fonts.css`, rely on `shared-globals.css`

Open LoneRifle opened this issue 3 years ago • 4 comments

What does this PR do?

styles/fonts.css is identical to shared-globals.css, as found in packages/ui/styles, and as referenced by styles/globals.css. Remove fonts.css and its references, leaning instead on the fonts described in shared-globals.css, which lines it up with other dependents of that file, like the Cal.com Storybook.

Screenshot below shows little change to download of fonts: image

Type of change

  • [x] Chore (refactoring code, technical debt, workflow improvements)
  • [x] This change requires a documentation update

How should this be tested?

  • Tested by visual inspection

LoneRifle avatar Dec 30 '22 08:12 LoneRifle

@LoneRifle is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Dec 30 '22 08:12 vercel[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Dec 30, 2022 at 6:04PM (UTC)

vercel[bot] avatar Dec 30 '22 17:12 vercel[bot]

@sean-brydon can you verify this and review?

PeerRich avatar Jan 01 '23 21:01 PeerRich

@JeroenReumkens - apologies - could you re-enable automerge (squash) and authorize Vercel to deploy? Thanks for the trouble =)

LoneRifle avatar Jan 02 '23 12:01 LoneRifle