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

REPL columnar -C #7969

Closed
scabug opened this issue Nov 13, 2013 · 5 comments
Closed

REPL columnar -C #7969

scabug opened this issue Nov 13, 2013 · 5 comments
Assignees
Milestone

Comments

@scabug
Copy link

@scabug scabug commented Nov 13, 2013

Completion results should be listed like ls -C by default instead of ls -x.

Provide a config property for those whose eyes scan -xily.

Column printing is done by jline, but we already override it, so customizing is a small tax. The flag would affect all columnar output, but maybe that domain includes only completions.

The subsidiary question is whether jline2 fixed the format problems that required overriding it.

@scabug
Copy link
Author

@scabug scabug commented Nov 13, 2013

Imported From: https://issues.scala-lang.org/browse/SI-7969?orig=1
Reporter: @som-snytt
Affected Versions: 2.11.0-M6

@scabug
Copy link
Author

@scabug scabug commented Nov 13, 2013

@som-snytt said:
I guess you can't convert a suggestion into an improvement?

@scabug
Copy link
Author

@scabug scabug commented Nov 15, 2013

@som-snytt said:
scala/scala#3147

Also adds variable column width.

@scabug
Copy link
Author

@scabug scabug commented Nov 18, 2013

Scaal (scaal) said:
Thank you for the patch. What is the difference between "suggestion" and "improvement" ?

@scabug
Copy link
Author

@scabug scabug commented Nov 18, 2013

@som-snytt said:
I don't know, but they have me trained to salivate only on "SI-" issues. In fact, "SI" stands for "salivate immediately," which was the message flashed on the screen while Beethoven's ninth played in the background. It's also possible that suggestions are subliminal, improvements are actionable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants