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.

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

Breadcrumbs: Should list ever be collapsed? #54491

Closed
JacksonKearl opened this issue Jul 17, 2018 · 9 comments
Closed

Breadcrumbs: Should list ever be collapsed? #54491

JacksonKearl opened this issue Jul 17, 2018 · 9 comments
Assignees
Milestone

Comments

@JacksonKearl
Copy link
Contributor

I'm not sure that it ever makes sense for the breadcrumbs to be focused but collapsed, gven Shift+Left/Right can navigate between layers even when the list is open. It seems that it just adds an additional step of Ctrl+Shift+. followed by Enter always. Am I missing something here?

@jrieken
Copy link
Member

jrieken commented Jul 17, 2018

I'm not sure that it ever makes sense for the breadcrumbs to be focused but collapsed, gven Shift+Left/Right can navigate between layers even when the list is open.

Not sure what you mean by collapsed? You mean not showing the picker?

@jrieken jrieken added info-needed Issue requires more information from poster breadcrumbs labels Jul 17, 2018
@JacksonKearl
Copy link
Contributor Author

Yes, is there utility in this view:
image
?

@jrieken
Copy link
Member

jrieken commented Jul 18, 2018

Yeah, it provides context about your current cursor position and allows you to navigate to them using ctrl/cmd-click and shift-enter. It has been explicitly been asked for
#9418 (comment). We can think of another keybinding the combines 'activate breadcrumbs' and 'select active crumb' tho

@mihailik
Copy link
Contributor

mihailik commented Jul 22, 2018

Breadcrumbs help making sense where you are in a large file.

After having them in Visual Studio (proper) it feels surprisingly difficult to read the same code in VSCode. TypeScript codebase itself has such large files - and scrolling to the middle you immediately left to guessing what is the context here.

That particularly applies to more functional languages like JavaScript - whereas in Java/C# you generally have very shallow nesting: class, then method. TypeScript/JavaScript often declare function within function, and the ambiguity of multiple closing curly brackets disorientates too much.

A navigating device like breadcrumbs helps a lot. Of course tuning the styles and some behaviour could make it even better, this is a prototype work in progress still. AWESOME work in progress!! :-D

@JacksonKearl
Copy link
Contributor Author

JacksonKearl commented Jul 23, 2018

What about if we allowed filtering by name while the list is not expanded, such that any typing would expand the list and start the filtering?

@mihailik
Copy link
Contributor

@JacksonKearl typing where?

@JacksonKearl
Copy link
Contributor Author

Typing when the breadcrubs bar has focus is currently a noop. It would be nice if it could start filtering as you type without needing to press Enter first

@mihailik
Copy link
Contributor

Yes, totally.

The keyboard story was completely off when I tried few days ago -- assuming it's left for later. Like, to close the dropdown you hit Esc like 5 times.

@jrieken jrieken removed the info-needed Issue requires more information from poster label Jul 25, 2018
@jrieken jrieken added this to the July 2018 milestone Jul 25, 2018
@jrieken
Copy link
Member

jrieken commented Jul 26, 2018

Pushed a change that makes Cmd+Shift+. focus and select the last breadcrumb. Also added another command Cmd+Shift+; that only focuses (as before)

@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants