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

Support request-body comments #11973

Closed
PhaedrusTheGreek opened this issue May 23, 2017 · 3 comments · Fixed by #133852
Closed

Support request-body comments #11973

PhaedrusTheGreek opened this issue May 23, 2017 · 3 comments · Fixed by #133852
Assignees
Labels
Feature:Console Dev Tools Console Feature Feature:Dev Tools high hanging fruit release_note:enhancement Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more

Comments

@PhaedrusTheGreek
Copy link
Contributor

Currently, the console does not allow comments in mid-body, but this feature would be highly useful.

screen shot 2017-05-23 at 11 52 46 am

It would also be great if you could collapse and comment.

screen shot 2017-05-23 at 11 53 51 am

@cjcenizal cjcenizal added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more and removed :Management DO NOT USE labels Jun 14, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui

@cjcenizal
Copy link
Contributor

cjcenizal commented Jun 10, 2020

Per @dgieselaar in #68770, we could also consider using // and /*...*/ (the latter for multiline comments) as these are supported by the JSON5 standard.

image

@cjcenizal
Copy link
Contributor

Per #64260, we'd also want to auto-indent comments when the auto-indent functionality is used. See that issue for an example of the desired behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Console Dev Tools Console Feature Feature:Dev Tools high hanging fruit release_note:enhancement Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more
Projects
None yet
6 participants