Minor fixes around playlist

[keikari]

Commit messages have details

The "Enter" being eaten issue was.

1. Go to video
2. Click "Save"
3. Click "New List" and type anything
4. The Enter is broken in comment creation, desc editing and search, until refresh.

Summary by CodeRabbit

• Bug Fixes
  • Collection names and titles are now automatically trimmed of whitespace during creation and publishing.
  • Improved error handling to ensure more reliable and consistent error messaging in collection-related operations.
  • Enhanced keyboard interaction handling for form submission, making the submission process more responsive and intuitive.

[coderabbitai]

Warning

Rate limit exceeded

@keikari has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 19 minutes and 52 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Three UI and action handler modifications improve data consistency and error handling: collection names are trimmed during creation and publication, a keyboard event handler is refactored for clarity, and error dispatches now safely handle undefined error properties with optional chaining fallbacks.

Changes

Cohort / File(s)	Summary
Input Trimming ui/component/formNewCollection/view.jsx, ui/page/collection/internal/collectionPublishForm/view.jsx	Trims collection names and titles during creation and publication by extracting trimmed values from form data and passing them through publish/edit workflows.
Event Handler Refactor ui/component/formNewCollection/view.jsx	Renames keyboard listener from altEnterListener to handleKeyDown and consolidates multiple focus/blur event bindings into a single onKeyDown handler on the input element.
Error Handling Safety ui/redux/actions/collections.js	Adds optional chaining guards for error.message access and implements fallback to raw error object when custom error messages are unavailable.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A little rabbit hops with glee,
Trimming names with precision and care,
Handlers renamed, events so rare,
Errors now caught with grace most fair,
Clean code flows through the spring air! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Minor fixes around playlist' is vague and generic, using non-descriptive language that doesn't convey specific information about the actual changes (trimming collection names, fixing Enter key behavior, improving error handling).	Revise the title to be more specific and descriptive of the main changes, such as 'Fix Enter key bug in playlist creation and add collection name trimming' or similar.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch playlist.issue

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@ui/component/formNewCollection/view.jsx`:
- Line 35: The button enablement uses the raw newCollectionName while
handleAddCollection trims it, allowing whitespace-only names to be submitted;
compute and reuse a trimmed variant (e.g., trimmedName =
newCollectionName.trim()) and use trimmedName for the button's disabled prop and
when creating the collection inside handleAddCollection (or guard early if
trimmedName.length === 0) so whitespace-only input is correctly treated as
empty; update references to newCollectionName in the button disabled check to
use that trimmed variable (and keep handleAddCollection using the same trimmed
value).

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Whitespace-only names bypass the disabled check.

newCollectionName is trimmed in handleAddCollection (Line 35), but the button's disabled prop on Line 79 checks newCollectionName.length === 0 against the untrimmed value. A user can type only spaces, the button stays enabled, and then an empty-string name is submitted.

Proposed fix

-            disabled={newCollectionName.length === 0}
+            disabled={newCollectionName.trim().length === 0}

Also applies to: 79-79

🤖 Prompt for AI Agents

In `@ui/component/formNewCollection/view.jsx` at line 35, The button enablement
uses the raw newCollectionName while handleAddCollection trims it, allowing
whitespace-only names to be submitted; compute and reuse a trimmed variant
(e.g., trimmedName = newCollectionName.trim()) and use trimmedName for the
button's disabled prop and when creating the collection inside
handleAddCollection (or guard early if trimmedName.length === 0) so
whitespace-only input is correctly treated as empty; update references to
newCollectionName in the button disabled check to use that trimmed variable (and
keep handleAddCollection using the same trimmed value).