dynamic-cli icon indicating copy to clipboard operation
dynamic-cli copied to clipboard

Syntax highlighting, Questions(along with the answers) and Proper line breaks

Open lakshyajit165 opened this issue 3 years ago • 7 comments

💥 Proposal

This is one of the results from a search in the dynamic-cli:

Screenshot from 2021-03-21 20-16-29

As you can see the syntax highlighting is not very prominent among a few other issues - so I would like to propose the following changes:

  • Better syntax highlighting (preferrably with a contrasting color)
  • If user searches for a query/phrase/keyword, it is better to include the questions along with answers as well - Right now only the answers' source links are provided, which might force the user from quitting the terminal and heading over to the site straightaway - which violates the main objective of this application.
  • Also as a user, I would like to have a proper demarcation(preferrably line breaks) between any 2 answers. Right now we are just providing a series of dashes(---).

Have you read the Contributing Guidelines on Pull Requests?

(Write your answer here.)

lakshyajit165 avatar Mar 21 '21 14:03 lakshyajit165

Sorry @lakshyajit165, We are using the rich module for rendering markdown. I don't think that is possible. Still we can try

lainq avatar Mar 21 '21 15:03 lainq

@pranavbaburaj if this rich module is open source we can try tweaking it acc. to our own requirements and then include that in this project.

lakshyajit165 avatar Mar 21 '21 15:03 lakshyajit165

@pranavbaburaj if this rich module is open source we can try tweaking it acc. to our own requirements and then include that in this project.

If that is the case, it will be good if you create the same issue in that repo - rich

lainq avatar Mar 21 '21 15:03 lainq

@pranavbaburaj Do you have any suggestion, If we can move to another module than rich?

GouravSardana avatar Mar 21 '21 16:03 GouravSardana

@pranavbaburaj Do you have any suggestions, If we can move to another module than rich?

I will try to find one

lainq avatar Mar 21 '21 17:03 lainq

You can try using terminal_markdown_viewer. This one looks promising in terms of syntax highlighting.

OBITORASU avatar Mar 31 '21 19:03 OBITORASU

@OBITORASU This looks really cool.

GouravSardana avatar Apr 01 '21 15:04 GouravSardana