Skip to content

feat: project name in title bar#1195

Merged
Kitenite merged 1 commit into
superset-sh:mainfrom
Ipriyankrajai:fix/1192
Feb 4, 2026
Merged

feat: project name in title bar#1195
Kitenite merged 1 commit into
superset-sh:mainfrom
Ipriyankrajai:fix/1192

Conversation

@Ipriyankrajai
Copy link
Copy Markdown
Contributor

@Ipriyankrajai Ipriyankrajai commented Feb 4, 2026

Description

Project name in title bar

Related Issues

#1192

Type of Change

Minor UI change
image

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

Testing

Screenshots (if applicable)

Additional Notes

Summary by CodeRabbit

  • New Features
    • The top bar now displays the workspace project name centered on the interface.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

The TopBar component was enhanced with a relative positioning class and a new absolutely-positioned overlay element that displays the workspace project name centered within the bar. The overlay uses pointer-events-none to prevent interaction interference and applies text truncation and muted styling.

Changes

Cohort / File(s) Summary
TopBar Layout Enhancement
apps/desktop/src/renderer/routes/_authenticated/_dashboard/components/TopBar/TopBar.tsx
Added relative positioning to TopBar container and introduced a centered, absolutely-positioned overlay displaying workspace project name with truncated text and muted styling.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A TopBar now stands tall and true,
With centered names in overlay's view,
Position set, pointer-events fade,
A careful balance, gracefully made! 🎯

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete. While it provides the issue reference (#1192) and a screenshot, it lacks detailed explanation of changes, testing steps, and properly filled checkboxes for type of change. Provide a more detailed description of the changes made, add testing verification steps, and properly select the type of change using checkboxes (likely 'New feature' or refactor the 'Other' section).
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.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding the project name to the title bar, matching the core functionality introduced in the changeset.

✏️ 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.

@iamhenry
Copy link
Copy Markdown

iamhenry commented Feb 4, 2026

@Ipriyankrajai @Kitenite what do you think of adding the branch name too?
[project] - [branch name] ex. openbox-deal-finder - feat/landing-page

I understand things could get busy but the intent here is to understand where you currently are. And if you're juggling a bunch of projects and a bunch of branches, quick visibility into that without having to navigate to another location would be really helpful.

@Kitenite
Copy link
Copy Markdown
Collaborator

Kitenite commented Feb 4, 2026

I think adding workspace name might help :). This is great!

@Kitenite
Copy link
Copy Markdown
Collaborator

Kitenite commented Feb 4, 2026

Will merge for cleanliness but feel free to follow up PR as well :)

@Kitenite Kitenite merged commit 8d72516 into superset-sh:main Feb 4, 2026
5 checks passed
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.

3 participants