Skip to content

Tree view: double-click to open process history only registers on the name cell, not the whole row #92

Description

@rezdm

Problem

The two view modes hit-test double-click differently:

  • Tree view (`render_process_tree_node`, src/ui/imgui/imgui_process_list_view.cpp:216): `if (ImGui::IsItemHovered() && ImGui::IsMouseDoubleClicked(0))` — checked right after `TreeNodeEx`, so `IsItemHovered()` only covers the tree-node's own item rect (the Process name cell + expand arrow), not the other columns.
  • List view (`render_process_list`, lines 488-509): computes a full-width row rectangle (`row_min.x`/`row_max.x` spanning the whole window) and checks the mouse position against that, so double-clicking anywhere in the row opens the popup.

So double-clicking on, say, the CPU% or PID column works in list mode but not tree mode — matches the reported "history on double click works on list mode only, not tree".

Fix

Tree mode already computes a full-row rect for its single-click handler (`row_y_min`/`row_y_max`, lines 223-226, used at line 276-284) — extend that same full-row hit-test to also drive the double-click-opens-popup behavior, matching list mode.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions