-
Notifications
You must be signed in to change notification settings - Fork 4k
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.
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
Ctrl+click GTD on a simple number argument creates link to type #23030
Comments
also look at #22786 |
I'm curious what they did want to happen. That said, ctrl-click navigation on literals seems unlikely for people to do. I'm guessing it was just fast-fingering that let to a suboptimal result. At least with symbols, people can see hte value. But the value for literals is so low that it would likely be better to just suppress things entirely than show a result that won't be appreciated. |
I don't see why we would treat literals for this feature any differently than we treat literals for other features:
I think we can treat this issue as a "learning curve" issue given we already accepted the muscle memory breaking change (Ctrl+Click for selecting a token is no longer usable, which may have led to the discovery of this issue but would not be restored by simply fixing this issue). |
per our design discussion today: Remove literal support from Ctrl+Click entirely (there will be no hyperlink when holding Ctrl and hovering over any literal). |
I'm reserving this for a new first time contributor.
Additional information:
|
I'd be interesting in grabbing this one, though I might not be able to start until this evening (juggling a newborn!). I'll get everything setup and jump back with the inevitable questions. |
@dwalleck Thanks Daryl! Please do not hesitate to reply to this thread if you run into any issues or need more information etc. |
Is that really the only file that has to be changed? |
@Neme12 It's not necessarily the only file that needs to change, but contains the central functionality relevant to the feature. Since the code base is very large, we like to provide a starting point for the investigation as a new contributor won't otherwise know where to look. |
I made this work by adding a parameter to |
@sharwell I'd love to give it a try but I'm a little discouraged by the fact that it takes me about 10 minutes to load the whole solution including all the package restores until VS becomes at least a little responsive, and even then being VS extremely slow, frequently freezing and sometimes just giving up entirely and hanging until I kill it manually, or the time it takes to build & run (I can't even dream of debugging) or the 90 minutes it took the tests to run from the command line. |
This is pretty much my focus, and I'm not the only one working to fix this. 😄 Here are some tips to improve the experience in the meantime:
If you have performance problems after the above steps, please let us know by following the steps here:
@jaredpar Does this sound normal for the current state of things? |
90 minutes from the command line is way too high. Should be around 13-15 minutes on a decent dev machine 20 on a slower one. Unit Tests should never drift anywhere near the 90 minute mark. |
@Neme12 How were you running the tests that took 90 minutes? |
@sharwell By running Is there a way to install the 15.6 preview as an update rather than side by side? |
Sorry I'm dragging on this. Who thought coding with a newborn around would be difficult? :-) I have a few questions that I'm trying to find the time to put together in sensible form. For the sake of expediency, if I can't get this knocked out in the next day or two would it make sense to let @Neme12 proceed with their solution? |
The first half of the tests takes 10 minutes and the rest takes at least another hour - they seem to get progressively slower. Is that normal? What's weird is that the test summary shows a run time of only about 12 minutes and a finish time an hour earlier (as if it only took 12 minutes). When running them, they take up so much CPU usage that the whole OS struggles to stay responsive, so the problem must be with me and not Visual Studio. I also get 2 failures related to localization, which is unexpected since I have everything on my machine set to en-US except for timezone & date formats, but the failure contains a localized error message for some reason, nothing related to date formatting... (why would it care about that setting?). I've never seen these localized messages and didn't know they existed. I even lie to my OS about my location being in the US. @dwalleck |
@dwalleck My suggestion is to take your time, but if you get to a point where you don't think you'll get to it then let us know. Currently @Neme12 is helping us with another issue (difficulty working in the project), plus we can always find new issues to help people get started whenever. The big advantage for these issues reserved for new contributors is removing pressure knowing there's no way a new user will |
@sharwell This one when loading a project but only when debugging and can be easily skipped:
but this one happens all over the place when I open a file and prevents me from doing anything:
and it brings me to some code that looks more like C-half#. This does not happen with my regular 15.5.5 installation (not even with my changes, they seem to be working correctly). |
Hi, I want to start contributing and getting familiar with the project. Is this a good starting task? |
@dwalleck Are you still working on this? @isc30 I would be glad to find you another starting item. Or if @dwalleck is not working on this then it would be good. While we wait for an answer, you can go through the instructions to make sure you have the code checked out and building on your machine: https://github.com/dotnet/roslyn/blob/dev15.7.x/docs/contributing/Building,%20Debugging,%20and%20Testing%20on%20Windows.md#developing-with-visual-studio-2017 😄 |
@sharwell Thanks a lot for the information! After following the steps I always get this Test result:
Is it normal? |
@isc30 If your default operating system language is something other than en-US, then than is currently expected, but we are working to fix it. It should still be possible to run individual tests from Test Explorer in Visual Studio 2017 version 15.6 Preview 4 and newer. |
@sharwell @JieCarolHu Sorry about the communication blackout. Picking up a new task with a newborn was bad planning on my part :-) If there's any other first timers issues in the future, I'd definitely be interested in picking one up |
Thanks @dwalleck , then I'm officially working on it! |
I managed to fix it. Some of the tests I introduced aren't working as expected:
I need some assistance to know if I'm going in the proper direction and some clue about why those tests are failing (with manual testing the cases are working well) |
change it to 15.8 as we are in escrow for 15.7 now |
…igation [#23030] Disable literal Ctrl+Click navigation
Reported by Mark Wilson-Thomas
Ctrl+Click Go To Definition on a simple number argument creates link to type! Not what I wanted...
The text was updated successfully, but these errors were encountered: