Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tweak Cmd+click navigation #2542

Closed
lukechurch opened this issue Apr 12, 2012 · 2 comments
Closed

Tweak Cmd+click navigation #2542

lukechurch opened this issue Apr 12, 2012 · 2 comments
Assignees
@stevemessick
Labels
closed-as-intended Closed as the reported issue is expected behavior

Comments

@lukechurch
Copy link
Contributor

Currently, if the cursor is over id, and the Cmd button is pressed, the underline navigation does not appear until the mouse is moved. It should appear straight away without any mouse movement being needed.
@clayberg
Copy link

Set owner to @stevemessick.

@stevemessick
Copy link
Contributor

I agree that would be nice and I've wondered why Eclipse doesn't do that. The answer is that activating hyperlinking on Cmd-key-down has bad interactions with other uses of Cmd. For example, if you are doing some copy/paste editing and you do it fast (using Cmd-C, Cmd-V) you suddenly find yourself editing a different file.

I don't think we can do this until we get sub-millisecond parse/resolve/analyze-types timings. Or at least sub-10ms.

Added AsDesigned label.

@lukechurch lukechurch added Type-Defect closed-as-intended Closed as the reported issue is expected behavior labels Apr 17, 2012
dart-bot pushed a commit that referenced this issue Aug 11, 2020
@sigurdm
Bump pub version
4b3db02 
> git log --oneline 04b054b62cc437cf23451785fdc50e49cd9de139..master
0d185a39 (HEAD -> master, origin/master, origin/HEAD) Push null-safety forwards to 2.11 (#2604)
56f9f27f Hide outdated --mode flag (#2603)
61ce6f81 Avoid double loop (#2605)
fa6e57d7 (disable_mixed_mode_validation) Fix outdated latest ordering (#2598)
6549e4aa Remove unused dependency from pubspec.yaml (#2592)
61543d07 Don't look for external package foo during testing (#2599)
590b448f Fixed license headers (#2595)
04e0601e Don't show entries for dev-dependencies in outdated --json --no-dev-dependencies (#2591)
8c3778c4 Configure GitHub move app (#2578)
eec7beca (top_level_command) Pass --(no-)sound-null-safety arg through to VM. (#2542)
152e4740 Warn about publishing in mixed mode (#2583)
0b7a3abe Removed april fools toys (#2325)
b74a5b73 Actually print hints (#2582)
8ec3a66d (pub2) Fix outdated --no-color (#2572)
7bb3d4e6 Use getSdkPath() in NullSafetyAnalysis (#2573)
3c578f24 Drop the "magic" package concept (#2577)
0e967ff0 Remove unused function (#2570)
988fefef Remove dependency overrides (#2568)

Change-Id: I58bf14234ed55bf9d825de60a40ded1d65281195
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/158003
Reviewed-by: Jonas Jensen <jonasfj@google.com>
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stevemessick stevemessick

Labels
closed-as-intended Closed as the reported issue is expected behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lukechurch @stevemessick @clayberg