Skip to content

feat: show breadcrumb context menu on mobile - #3497

Merged
AlexAndBear merged 2 commits into
mainfrom
feat/breadcrumb-context-menu-mobile
Sep 30, 2026
Merged

AlexAndBear merged 2 commits into
mainfrom
feat/breadcrumb-context-menu-mobile

Conversation

@AlexAndBear

@AlexAndBear AlexAndBear commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Description

The context menu ("three dots") of the current breadcrumb item was only available in the desktop breadcrumb. On mobile, users had no way to access the actions for the current folder from the breadcrumb.

This PR renders OcBreadcrumbContextMenu next to the current folder in the mobile breadcrumb as well (.oc-breadcrumb-mobile-current), using the same showContextActions condition and contextMenuPadding as on desktop. Each OcBreadcrumbContextMenu instance generates unique trigger/drop IDs, so the desktop and mobile instances don't conflict. The inactive variant is hidden via the existing Tailwind breakpoint classes.

The context menu drop now also uses the current folder name as its title, which is shown in the bottom drawer on mobile.

Related Issue

  • Fixes <issue_link>

How Has This Been Tested?

  • test environment: Vitest unit tests
  • test case 1: context menu is rendered for the last desktop list item when showContextActions is enabled
  • test case 2: context menu is rendered in the mobile current item when showContextActions is enabled
  • test case 3: both context menu instances use the current folder name as title
  • test case 4: context menu is not rendered when showContextActions is disabled

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@AlexAndBear
AlexAndBear merged commit 7abb942 into main Sep 30, 2026
31 checks passed
@AlexAndBear
AlexAndBear deleted the feat/breadcrumb-context-menu-mobile branch September 30, 2026 11:40
openclouders pushed a commit that referenced this pull request Sep 30, 2026
* feat: show breadcrumb context menu on mobile

* feat: use current folder name as breadcrumb context menu title
@openclouders openclouders mentioned this pull request Sep 30, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants