Skip to content

fix: remove trailing whitespace from fonts.ts#3112

Open
guangyang1206 wants to merge 1 commit into
onlook-dev:mainfrom
guangyang1206:fix/remove-trailing-whitespace-fonts
Open

fix: remove trailing whitespace from fonts.ts#3112
guangyang1206 wants to merge 1 commit into
onlook-dev:mainfrom
guangyang1206:fix/remove-trailing-whitespace-fonts

Conversation

@guangyang1206
Copy link
Copy Markdown

@guangyang1206 guangyang1206 commented May 23, 2026

Remove trailing whitespace for code hygiene.

Fixes #3081

Description

Problem

Line 6 of apps/web/client/src/app/fonts.ts had a trailing whitespace character, which triggers warnings in editors and CI linting tools that flag trailing spaces.

Fix

Removed the trailing whitespace on line 6 of apps/web/client/src/app/fonts.ts.

Testing

  • No functional change — this is a code hygiene / whitespace cleanup only
  • Verified the file renders identically; no runtime behavior is affected

Fixes #3081

Summary by CodeRabbit

  • Style
    • Fixed minor formatting in font configuration.

Review Change Stack

Remove trailing whitespace for code hygiene.

Fixes onlook-dev#3081
@vercel
Copy link
Copy Markdown

vercel Bot commented May 23, 2026

@guangyang1206 is attempting to deploy a commit to the Onlook Team on Vercel.

A member of the Team first needs to authorize it.

@vercel vercel Bot temporarily deployed to Preview – docs-onlook May 23, 2026 09:29 Inactive
@vercel
Copy link
Copy Markdown

vercel Bot commented May 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs-onlook Skipped Skipped May 23, 2026 9:29am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c5edf443-b02e-4a5f-8b52-50210a2bda3d

📥 Commits

Reviewing files that changed from the base of the PR and between a242be5 and 974e50f.

📒 Files selected for processing (1)
  • apps/web/client/src/app/fonts.ts

📝 Walkthrough

Walkthrough

This pull request removes trailing whitespace from line 7 of apps/web/client/src/app/fonts.ts. The change affects only the closing bracket of the Vujahday_Script configuration used in the vujahdayScript export, with no functional modifications.

Changes

Whitespace Cleanup

Layer / File(s) Summary
Fonts configuration whitespace cleanup
apps/web/client/src/app/fonts.ts
Trailing whitespace is removed from the closing }); of the Vujahday_Script configuration for code hygiene.

🎯 1 (Trivial) | ⏱️ ~1 minute

🐰 A trailing space did roam,

In fonts where scripts call home,

With whisker and care,

We cleaned the air,

Now perfect formatting's shown! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: removing trailing whitespace from fonts.ts.
Description check ✅ Passed The PR description covers the problem, fix, and testing; however, it does not use the repository's template structure with sections like Type of Change and proper formatting.
Linked Issues check ✅ Passed The PR successfully addresses the objective from issue #3081: removing trailing whitespace from fonts.ts for code hygiene.
Out of Scope Changes check ✅ Passed The changes are narrowly scoped to removing trailing whitespace from fonts.ts, directly aligned with the linked issue requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: private package registry requires authentication. Disable ESLint in CodeRabbit settings or use public packages.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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