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

make auto completion path short #17357

Closed
wants to merge 1 commit into from

Conversation

AlexHolly
Copy link
Contributor

#4704

Still not a good solution I think there is a horizontal scoll missing.

peek 2018-03-08 01-38

@akien-mga
Copy link
Member

I agree about the need to enhance this, but I'm not convinced by the proposed fix, I don't find it readable with all path parts cut in 3 with the middle part replaced by a simple dot.

It would be better to cut in the middle of the whole path with three dots, as proposed in #4704 originally.
Or yes, even better would be to have horizontal scrolling of the hovered line to avoid having to cut it.

Or actually both: ellipsis when not hovered (so that it fits in the dialog), and when hovered the ellipsis is removed and the full path scrolls instead. So adding only the ellipsis for now would be a first step.

@akien-mga
Copy link
Member

As per the above, I'm not convinced by this proposal, so closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants