Skip to content

fix(desktop): make right-sidebar refresh button always visible instead of hover-reveal - #45450

Closed
XueQi-2957 wants to merge 2 commits into
NousResearch:mainfrom
XueQi-2957:fix/right-sidebar-refresh-btn-always-visible
Closed

XueQi-2957 wants to merge 2 commits into
NousResearch:mainfrom
XueQi-2957:fix/right-sidebar-refresh-btn-always-visible

Conversation

@XueQi-2957

Copy link
Copy Markdown

Bug Description

The refresh button in the file explorer's top bar (right-sidebar) was only visible when hovering over the folder-name label. As the user moved their mouse from the label toward the button, they left the label area — causing the button to disappear before they could click it, making the refresh button effectively unusable.

Root Cause

The refresh button used HEADER_ACTION_LABEL_REVEAL CSS classes:

pointer-events-none opacity-0 transition-opacity
peer-hover/project-label:pointer-events-auto peer-hover/project-label:opacity-100

This made the button invisible (opacity-0) and non-interactive (pointer-events-none) by default. It only became visible when the adjacent folder-name label (.peer/project-label) was hovered — but moving the cursor toward the button left the label, removing the peer-hover state and hiding the button again.

Fix

  1. Change the refresh button's className from HEADER_ACTION_LABEL_REVEAL to HEADER_ACTION_CLASS — the same always-visible class used by the open-folder and collapse-all buttons.
  2. Remove the now-unused HEADER_ACTION_LABEL_REVEAL constant and update the comment.

How to Verify

  1. Open Hermes Desktop with a project folder selected.
  2. Look at the right-sidebar's top bar — the refresh button (⟳) should always be visible next to the folder name.
  3. Move your mouse directly onto the refresh button and click it — it should refresh the file tree without disappearing.

Risk Assessment

Low — only a CSS class change on one button. The other two toolbar buttons (open-folder, collapse-all) already use HEADER_ACTION_CLASS with no issues. The worst case is the button remains visible when it shouldn't be, which is the desired behavior.

The refresh button in the file tree's section header used a hover-reveal
pattern (HEADER_ACTION_LABEL_REVEAL): it was opacity-0 + pointer-events-none
by default and only appeared when hovering the adjacent folder-name label
(peer/project-label). This created a usability deadlock: as the user moved
their mouse from the label toward the button, they left the label — the
peer-hover CSS stopped applying — causing the button to disappear before
the cursor could reach it.

Fix: use the same always-visible HEADER_ACTION_CLASS as the other toolbar
buttons (open-folder, collapse-all). Remove the now-unused
HEADER_ACTION_LABEL_REVEAL constant.

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Summary

Verdict: Approved

UX improvement: the sidebar refresh button is now always visible instead of requiring hover on desktop. Small but meaningful usability fix for the right panel.

Looks Good

  • Single CSS/styling change (hover-reveal → always-visible)
  • No security concerns, no test changes needed

@alt-glitch alt-glitch added type/bug Something isn't working comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint P3 Low — cosmetic, nice to have labels Jun 13, 2026

@XueQi-2957 XueQi-2957 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

fine

@teknium1

Copy link
Copy Markdown
Collaborator

Thanks for the focused desktop usability report and patch.

Automated hermes-sweeper review found that current main already implements the reported reachability guarantee:

Closing as implemented on main.

@teknium1 teknium1 closed this Jul 14, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 14, 2026
@XueQi-2957
XueQi-2957 deleted the fix/right-sidebar-refresh-btn-always-visible branch July 14, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants