fix: remove settings option from stream-view #277

2026-06-11T23:25:34Z

talksik commented

2026-06-11 23:25:34 +00:00

(Migrated from github.com)

Summary by CodeRabbit

Changes
- Removed the "Settings" option from the stream top-bar dropdown menu.
- Stream top-bar menu simplified—fewer actions displayed in the dropdown.
- Access to settings remains available through other parts of the app, so workflow for managing preferences is unchanged.

## Summary by CodeRabbit * **Changes** * Removed the "Settings" option from the stream top-bar dropdown menu. * Stream top-bar menu simplified—fewer actions displayed in the dropdown. * Access to settings remains available through other parts of the app, so workflow for managing preferences is unchanged.

coderabbitai[bot] commented

2026-06-11 23:25:49 +00:00

(Migrated from github.com)

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 541a80ab-e111-4f06-bee4-88c633aed0d5

📥 Commits

Reviewing files that changed from the base of the PR and between 202c7180da and 6ebf4abc72.

📒 Files selected for processing (1)

js/desktop/src/features/particles/stream-top-bar.tsx

💤 Files with no reviewable changes (1)

js/desktop/src/features/particles/stream-top-bar.tsx

📝 Walkthrough

Walkthrough

The stream top-bar actions dropdown no longer includes a Settings entry; the Settings icon import, the useNavigate hook usage, and the dropdown item linking to /settings were removed from the component.

Changes

Stream Top Bar Settings Removal

Layer / File(s)	Summary
Remove Settings menu item `js/desktop/src/features/particles/stream-top-bar.tsx`	Settings icon import from `lucide-react` is removed, `useNavigate` and its initialization are deleted, and the dropdown menu item navigating to `/settings` is removed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops past code tonight,
A cog removed, the menu light,
No gear to turn, no path to roam,
The top bar finds a quieter home. 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing the settings option from the stream-view dropdown menu, which is clearly reflected in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch cleanup-stream-view-options

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

[![Review Change Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/flowy-live/llink/pull/277?utm_source=github_walkthrough&utm_medium=github&utm_campaign=change_stack)  No actionable comments were generated in the recent review. 🎉 <details> <summary>ℹ️ Recent review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro Plus **Run ID**: `541a80ab-e111-4f06-bee4-88c633aed0d5` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 202c7180dacf8ec646b82e7bf6eea6e34139191b and 6ebf4abc72ca759385dc3edb4911ca48293d5826. </details> <details> <summary>📒 Files selected for processing (1)</summary> * `js/desktop/src/features/particles/stream-top-bar.tsx` </details> <details> <summary>💤 Files with no reviewable changes (1)</summary> * js/desktop/src/features/particles/stream-top-bar.tsx </details> </details> ---  <details> <summary>📝 Walkthrough</summary> ## Walkthrough The stream top-bar actions dropdown no longer includes a Settings entry; the Settings icon import, the `useNavigate` hook usage, and the dropdown item linking to `/settings` were removed from the component. ## Changes **Stream Top Bar Settings Removal** |Layer / File(s)|Summary| |---|---| |**Remove Settings menu item** `js/desktop/src/features/particles/stream-top-bar.tsx`|Settings icon import from `lucide-react` is removed, `useNavigate` and its initialization are deleted, and the dropdown menu item navigating to `/settings` is removed.| ## Estimated code review effort 🎯 1 (Trivial) | ⏱️ ~2 minutes ## Poem > A rabbit hops past code tonight, > A cog removed, the menu light, > No gear to turn, no path to roam, > The top bar finds a quieter home. 🐰✨ </details>   <details> <summary>🚥 Pre-merge checks | ✅ 5</summary> <details> <summary>✅ Passed checks (5 passed)</summary> | Check name | Status | Explanation | | :------------------------: | :------- | :------------------------------------------------------------------------------------------------------------------------------------------------------------ | | Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. | | Title check | ✅ Passed | The title accurately describes the main change: removing the settings option from the stream-view dropdown menu, which is clearly reflected in the changeset. | | Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. | | Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. | | Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. | </details> ✏️ Tip: You can configure your own custom pre-merge checks in the settings. </details>   <details> <summary>✨ Finishing Touches</summary> <details> <summary>📝 Generate docstrings</summary> - [ ]  Create stacked PR - [ ]  Commit on current branch </details> <details> <summary>🧪 Generate unit tests (beta)</summary> - [ ]  Create PR with unit tests - [ ]  Commit unit tests in branch `cleanup-stream-view-options` </details> </details>   --- Comment `@coderabbitai help` to get the list of available commands and usage tips.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FlowyLabs/llink#277