Page MenuHomePhorge

Fix middle-click, CTRL+click, right-click etc. on Typehead search results
ClosedPublic

Authored by valerio.bozzolan on Feb 26 2023, 21:51.
Tags
None
Referenced Files
F2648015: D25069.1732387560.diff
Fri, Nov 22, 18:46
F2647389: D25069.1732366866.diff
Fri, Nov 22, 13:01
F256610: D25069 after-before demonstration.mp4
Feb 27 2023, 18:21
Tokens
"Love" token, awarded by 20after4."Cup of Joe" token, awarded by valerio.bozzolan."Yellow Medal" token, awarded by avivey.

Details

Summary

Fix middle-click, CTRL+click, right-click etc. on Typehead search results.

Closes T15149

Test Plan

Try the following actions on various typeheads:

  • right-click
  • middle-click
  • CTRL+click
  • normal click

Demonstration video (2M) showing After patch (on localhost) and Before patch (here on we.phorge.it), where I middle-click and normal-click on menu entries:

Notes:

  • the middle click now works (opening in new tab)
  • the CTRL+click (or "command" key + click) now works (opening in new tab)
  • the right click now opens the context menu (previously broken)
  • the normal click should just click (as usual)

Try on:

  • search results while typing in main search bar
  • search results when editing a Task Tags / assigned to, etc.
  • try to click on other weird places
  • $$$

Diff Detail

Repository
rP Phorge
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 119
Build 119: arc lint + arc unit

Event Timeline

rollback last changes made by mistake using arc diff

avivey awarded a token.
This revision is now accepted and ready to land.Mar 3 2023, 10:24