Skip to content

Fix typo in comments layout description #724

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
justin808 wants to merge 1 commit into from
Closed

Fix typo in comments layout description #724

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion client/app/bundles/comments/layout/Layout.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export default class Layout extends Component {
</p>
<h2 className="mt-3 text-3xl text-slate-900 sm:text-4xl">The same comments app, with routing.</h2>
<p className="mt-3 max-w-2xl text-sm leading-7 text-slate-600 sm:text-base">
Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
Sasah, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
Copy link
Copy Markdown

@cursor cursor bot Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Accidentally committed text in UI description

Medium Severity

The string "Sasah, " was prepended to the user-facing description text. This appears to be accidentally committed personal text (possibly a name or test string) rather than an intentional typo fix. It will be visible to all users in the rendered UI.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8aed2f3. Configure here.

Copy link
Copy Markdown

@claude claude bot Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change introduces a problem rather than fixing one. "Sasah, " appears to be an accidentally committed name/word that doesn't belong in UI copy. The original text was grammatically correct and clear.

Suggested change
Sasah, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
Use the pills to view the comments route, a plain routed screen, and a redirect back to root.

Copy link
Copy Markdown

@greptile-apps greptile-apps bot Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Spurious word introduces a typo instead of fixing one

The word Sasah, prepended to the sentence has no meaning here and is not a valid English word in this context — the PR title says "Fix typo" but this change actually introduces one. The original text was correct and should be restored.

Suggested change
Sasah, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
Use the pills to view the comments route, a plain routed screen, and a redirect back to root.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot Apr 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix the newly introduced user-facing typo (Sasah → Sasha).

Line 28 appears to introduce a typo in the demo description copy, which conflicts with the typo-fix intent of this PR.

Suggested patch 
-              Sasah, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
+              Sasha, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Sasah, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
Sasha, Use the pills to view the comments route, a plain routed screen, and a redirect back to root.
🧰 Tools
🪛 ESLint

[error] 28-28: Insert ⏎·············

(prettier/prettier)

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@client/app/bundles/comments/layout/Layout.jsx` at line 28, The displayed demo
copy in the Layout component contains a user-facing typo "Sasah" that should be
corrected to "Sasha"; locate the string in the Layout.jsx (the JSX output/render
of the Layout component) and update the text content so the sentence reads
"Sasha, Use the pills to view the comments route, a plain routed screen, and a
redirect back to root." ensuring only the typo is fixed and surrounding
punctuation/spacing is preserved.

</p>
</div>

Expand Down
Loading