fix(desktop): update TopBar to display project and workspace names together

[Ipriyankrajai]

Description

Related Issues

Type of Change

• Bug fix
• New feature
• Documentation
• Refactor
• Other (please describe):

Testing

Screenshots (if applicable)

Additional Notes

Summary by CodeRabbit

• New Features
  • The top bar project title now displays both workspace and project names for improved context visibility, with expanded display width to accommodate the combined information.

[coderabbitai]

📝 Walkthrough

Walkthrough

The TopBar component now displays both the workspace and project names combined as "workspace.project.name - workspace.name" instead of just the project name. The maximum width constraint for this centered title was increased from 200px to 300px to accommodate the longer text.

Changes

Cohort / File(s)	Summary
TopBar Display Update apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/TopBar/TopBar.tsx	Updated centered title to concatenate workspace and project names with " - " separator; increased max-width from 200px to 300px for improved visibility.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• feat: project name in title bar #1195: Modifies the same TopBar.tsx file to add an absolutely positioned overlay for the project name, which may conflict or relate to this title display change.
• feat(desktop): add workspace settings section with project/workspace navigation #247: Adds the workspace.project field to the workspace object returned from the backend, which is the data source dependency for this PR's new display logic.

Poem

🐰 A TopBar tale, I must confess,
Now shows both names in one address,
Project and workspace, side by side,
With wider space, they both fit snide!
More room to read, less need to hide! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

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.
Description check	⚠️ Warning	The PR description only includes the template structure with placeholder comments but lacks substantive implementation details.	Add a clear explanation of what changed and why (e.g., 'Updated TopBar to display project and workspace names together'), reference any related issues, select the change type, describe testing performed, and add screenshots showing the UI changes.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and accurately summarizes the main change: updating TopBar to display project and workspace names together, matching the code modification that concatenates workspace.name with workspace.project.name.

_{✏️ 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[Ipriyankrajai]

@Kitenite
@iamhenry
#1192

[iamhenry]

❤️

[Kitenite]

Lovely, thank @Ipriyankrajai !

[iamhenry]

I just noticed this

is that in superset already?? @Kitenite