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

GUI: parse Deskew CLI output #27

Open
galfar opened this issue Jun 11, 2020 · 0 comments
Open

GUI: parse Deskew CLI output #27

galfar opened this issue Jun 11, 2020 · 0 comments
Labels

Comments

@galfar
Copy link
Owner

galfar commented Jun 11, 2020

Newer GUI version may not work correctly when paired with outdated CLI program (new options, deprecated options, etc.) => deskewing then fails but there is just not very visible e.g. "Invalid params" message in the output log.

GUI could check if the CLI is a reasonable version (parse the captured console output).
This could be printed somewhere on the "Progress" page (Detected version: 1.40, tested version: 1.35). Or in the console output with some highlight.

Another possibility is to color the console output (line background?) when Error or FinishedOk message is detected. We would have to switch from TMemo to TRichMemo, such styling is supposed to be supported on all 3 supported platforms: https://wiki.freepascal.org/RichMemo#Append_mixed_color_text_at_the_end_of_the_RichMemo
Colors:

  • error: reddish
  • finished ok: greenish
  • useful info (in + out file names): light blueish
@galfar galfar added enhancement New feature or request DeskewGui lower prio labels Jun 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant