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
F14310611: D13550.id32739.diff
Wed, Dec 18, 1:45 AM
Unknown Object (File)
Mon, Dec 16, 5:14 PM
Unknown Object (File)
Sun, Dec 15, 2:00 PM
Unknown Object (File)
Sat, Dec 14, 1:44 PM
Unknown Object (File)
Thu, Dec 12, 9:12 AM
Unknown Object (File)
Wed, Dec 11, 9:00 PM
Unknown Object (File)
Mon, Dec 9, 7:45 PM
Unknown Object (File)
Mon, Dec 9, 3:03 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
resources/
celerity/
18 lines
webroot/
rsrc/
js/
application/
repository/
2 lines

Diff 33828

View Options

resources/celerity/map.php

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility