Skip to content

Conversation

@anmolsinghbhatia
Copy link
Collaborator

@anmolsinghbhatia anmolsinghbhatia commented Dec 24, 2025

Description

This PR improves the sidebar list active project indicator.

Type of Change

  • Improvement

Summary by CodeRabbit

  • Style
    • Improved sidebar project highlighting: the selected project is now visually emphasized when the URL indicates that project and navigation is not in accordion mode. Visual update only — no interaction or drag/drop behavior changed.
    • Updated divider appearance inside open-project panel for clearer separation.

✏️ Tip: You can customize this high-level summary in your review settings.

@anmolsinghbhatia anmolsinghbhatia self-assigned this Dec 24, 2025
Copilot AI review requested due to automatic review settings December 24, 2025 14:14
@makeplane
Copy link

makeplane bot commented Dec 24, 2025

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 24, 2025

Caution

Review failed

The pull request is closed.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

📝 Walkthrough

Walkthrough

Added a conditional highlight so a project list item gets bg-layer-transparent-active when the current URL project ID equals the item's project ID and navigation mode is not "accordion". Only styling and a divider class token were changed; no logic or event handling was modified.

Changes

Cohort / File(s) Summary
Projects list item (styling)
apps/web/core/components/workspace/sidebar/projects-list-item.tsx
Added shouldHighlightProject derived boolean; apply bg-layer-transparent-active to the item container when URL project id matches the item's id and navigation mode !== "accordion"; changed a divider class from bg-subtle-1 to bg-layer-3. No changes to drag/drop or event handlers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

I nibble lines and tweak a hue,
Where URL and item meet true,
A quiet glow beneath my paw,
Small change, big cheer — hip hip hoorah! 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes only 2 of 6 template sections: Description and Type of Change. Critical sections like Test Scenarios, Screenshots, and References are missing. Complete the PR description by adding Test Scenarios, Screenshots (if applicable), and References sections to match the repository template.
Title check ❓ Inconclusive The title mentions 'sidebar project list enhancements' which is vague and doesn't specifically describe the main change—improving the active project indicator highlight logic. Make the title more specific about the actual change, e.g., 'Fix sidebar project highlight based on current URL' or 'Improve active project indicator in sidebar'.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c39f0bd and f274d4c.

📒 Files selected for processing (1)
  • apps/web/core/components/workspace/sidebar/projects-list-item.tsx

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the sidebar project list by adding a visual indicator for the currently active project. The active project is now highlighted with a distinct background color (bg-layer-transparent-active) when its ID matches the URL project parameter.

Key Changes

  • Added conditional styling to highlight the active project in the sidebar based on URL project ID match

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sriramveeraghanta sriramveeraghanta merged commit 55c49cc into preview Dec 24, 2025
8 of 9 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore-sidebar-project-list-enhancements branch December 24, 2025 14:23
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.

4 participants