-
Notifications
You must be signed in to change notification settings - Fork 29.5k
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
Very difficult to find file's path with tabs #8501
Comments
I like your point @alexandrudima about making hover on the whole tab. I, too, found it a little awkward because I had to hit the right target, and it's not clear where the text's vertical boundaries are. @bpasero is this doable? As far as the timing, how about we first make that hover change and see if that fixes the pain? It may no longer be an issue since the hover target will be so easy to hit accurately. I worry decreasing the delay with such a large hover area may make this hover tip appear more than is desirable. |
I raised this earlier here #7960, wanting to be able to get at the path using just the keyboard. |
The hover is native and the timing cannot be changed. Introducing a custom hover is possible but we do this nowhere else. I have no good solution for this other than suggesting to disable tabs if you really need the path to be visible. |
There is actually an extension that just puts the full path of the file to the status bar: https://marketplace.visualstudio.com/items?itemName=RoscoP.ActiveFileInStatusBar |
@bpasero that's good enough for me 👍 |
That seems like a fine solution. In fact, I was convinced that we had designed that in our original mockups but it turns out we didn't. We should consider just doing this I think. |
I like to keep this up for an extension, if we did this out of the box, many people might find it annoying and ask for a way to disable. |
If this turns out to be a problem for only a few people then I'm fine with an extension. But if this turns out to be a problem for a large number of people then we really should reconsider and figure out how to solve this out of the box. |
Btw I still think we should maybe look into a custom hover if the native hover is so hard to be used. But we can leave this closed until we get more feedback from users once June is out 👍 |
How does this extension target my specific two points above:
I understand point 1 is non-trivial as we now use the native hover. |
@alexandrudima sorry, those are good points, I will open a separate issue on the hover thing. |
@bpasero thank you 👍 |
Testing #7854
The text was updated successfully, but these errors were encountered: