Skip to content

Conversation

@malik-jouda
Copy link
Contributor

@malik-jouda malik-jouda commented Feb 1, 2026

πŸ”— Linked issue

Resolves #5976

❓ Type of change

Before

Screenshot 2026-02-01 at 11 16 22β€―PM

After

Screenshot 2026-02-01 at 11 15 28β€―PM
  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@github-actions github-actions bot added the v4 #4488 label Feb 1, 2026
@dosubot
Copy link

dosubot bot commented Feb 1, 2026

Related Documentation

No published documentation to review for changes on this repository.

Write your first living document

How did I do? Any feedback?Β Β Join Discord

@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

πŸ“ Walkthrough

Walkthrough

This change replaces two CSS utility class names in the editor theme configuration file. Specifically, float-left is replaced with float-start in the placeholder styling for both placeholderMode.firstLine.base and placeholderMode.everyLine.base. No logic, control flow, or public API signatures are affected by this update.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

πŸš₯ Pre-merge checks | βœ… 3
βœ… Passed checks (3 passed)
Check name Status Explanation
Title check βœ… Passed The title accurately describes the main change: replacing float-left with float-start to handle placeholder styling in RTL mode.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check βœ… Passed The pull request description is minimal but related to the changesetβ€”it links to issue #5976 about RTL placeholder handling in the Editor component, which aligns with the CSS class changes from float-left to float-start in placeholder styling.

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

✨ Finishing touches
  • πŸ“ Generate docstrings
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 1, 2026

npm i https://pkg.pr.new/@nuxt/ui@5977

commit: 1b729ae

Copy link
Member

@benjamincanac benjamincanac left a comment

Choose a reason for hiding this comment

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

Thanks! 😊

@benjamincanac benjamincanac merged commit 3cc16e3 into nuxt:v4 Feb 2, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v4 #4488

Projects

None yet

Development

Successfully merging this pull request may close these issues.

wrong direction for Editor placeholder on rtl

2 participants