Skip to content

fix: skip home bookmarks in the bookmark listings - #9952

Merged
nishantmonu51 merged 1 commit into
mainfrom
bookmark-naming
Sep 24, 2026
Merged

nishantmonu51 merged 1 commit into
mainfrom
bookmark-naming

Conversation

@nishantmonu51

Copy link
Copy Markdown
Collaborator
  • Home bookmarks are no longer listed on the project home page or the all-bookmarks page. Opening a dashboard already shows its home view, so a home row only duplicated the dashboard's own link, and with many dashboards the list filled up with rows reading "Go to Home" or "Home".
  • Removes the now unreachable home branches: the home category, the Home tag and icon in BookmarksTableCompositeCell, the home guards in BookmarkMetadataDialog, and the unused bookmark_tag_home message.
  • Home bookmarks are still created, opened and deleted from the home menu on the dashboard.

Context: Slack thread

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

Opening a dashboard already shows its home view, so a home bookmark row only duplicated the dashboard's own link and read as a meaningless "Go to Home" or "Home" entry when a project has many dashboards.
@nishantmonu51
nishantmonu51 merged commit 02ab0f0 into main Sep 24, 2026
17 checks passed
@nishantmonu51
nishantmonu51 deleted the bookmark-naming branch September 24, 2026 21:51
nishantmonu51 added a commit that referenced this pull request Sep 24, 2026
Opening a dashboard already shows its home view, so a home bookmark row only duplicated the dashboard's own link and read as a meaningless "Go to Home" or "Home" entry when a project has many dashboards.
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.

2 participants