Page MenuHomePhabricator
Differential D13550

Symbol Search: Allow ctrl-click with no hover
ClosedPublic
Actions

Authored by avivey on Jul 4 2015, 3:19 AM.
Tags
None
Referenced Files
F19058555: D13550.id33828.diff
Sat, Nov 29, 2:32 AM
F19058554: D13550.id33828.diff
Sat, Nov 29, 2:32 AM
F18870782: D13550.id32737.diff
Tue, Nov 4, 6:58 PM
F18840477: D13550.id32737.diff
Oct 28 2025, 1:12 AM
F18757448: D13550.id33828.diff
Oct 5 2025, 5:28 PM
F18621639: D13550.diff
Sep 15 2025, 9:01 AM
F18048664: D13550.id.diff
Aug 3 2025, 1:41 PM
F17927843: D13550.id32737.diff
Jul 30 2025, 3:17 PM
View All Files
Subscribers
epriestley
gena2x
joshuaspence
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8710: Symbol lookup requires ctrl-hover-and-click, instead of just ctrl-click
Commits
Restricted Diffusion Commit
rP328d336c8bb3: Symbol Search: Allow ctrl-click with no hover
Summary

Fix T8710. I had hopes of doing something cleaver with highlighted (Like trying to understand foo.bar when clicking bar, but I obviously didn't do it.

Test Plan

ctrl-click.

Diff Detail

Repository
rP Phabricator
Branch
ctrl-click
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7139
Build 7326: [Placeholder Plan] Wait for 30 Seconds
Build 7325: arc lint + arc unit

Event Timeline

avivey updated this revision to Diff 32737.Jul 4 2015, 3:19 AM
avivey retitled this revision from to Symbol Search: Allow ctrl-click with no hover.
avivey updated this object.
avivey edited the test plan for this revision. (Show Details)
avivey added reviewers: epriestley, joshuaspence.
avivey added a task: T8710: Symbol lookup requires ctrl-hover-and-click, instead of just ctrl-click.
avivey added a subscriber: gena2x.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 4 2015, 3:19 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
avivey updated this object.Jul 4 2015, 3:20 AM
avivey edited edge metadata.
avivey updated this revision to Diff 32739.Jul 4 2015, 4:21 AM

celerify

joshuaspence resigned from this revision.Jul 7 2015, 1:14 PM
joshuaspence removed a reviewer: joshuaspence.

I don't know enough about Phabricator JavaScript to comment here.

joshuaspence added a subscriber: joshuaspence.Jul 19 2015, 12:46 AM
avivey mentioned this in Z1336: General Chat.Aug 27 2015, 2:31 AM
epriestley accepted this revision.Aug 27 2015, 11:17 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 27 2015, 11:17 AM
Closed by commit rP328d336c8bb3: Symbol Search: Allow ctrl-click with no hover (authored by avivey, committed by epriestley). · Explain WhyAug 27 2015, 11:17 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
webroot/
rsrc/
js/
application/
repository/
2 lines

Diff 32737

View Options

webroot/rsrc/js/application/repository/repository-crossreference.js

Loading...
Phabricator · Built by Phacility