-
Notifications
You must be signed in to change notification settings - Fork 8
inital Txt viewer #40
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
base: main
Are you sure you want to change the base?
Conversation
… depends on canvas setup
|
Also, I think application should:
txt-files for the testing can be found here test-txts.zip.
|
about this part, we've a sort of toolbar at bottom area, meant to display some maybe minimal progress, some additional data. is evaluated inside draw method and yeh, if line don't fit, it won't be displayed, though it looks a bit worse than expected, maybe I wasn't that precise, would check |
|
other 2 points, 2., I guess that won't be really hard, with binary 've to do more than those, |
|
anyways, thanks for feedback, would try to improve it :) |
…d some other small improvements
…ementation of callback for txtviewer
|
|
Most of the test files works perfect 🙂 But. |
… readjust block sizes
|
If user tries to open empty file ('empty.txt'), application crashes and Keira going to reboot. Decoded backtrace: |
| } | ||
|
|
||
| String TxtViewerApp::onGetStrFontSize() { | ||
| return StringFormat("Font SIZE %d C(-) A(+)", fontSize); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be "D(-)".
| return StringFormat("Font SIZE %d C(-) A(+)", fontSize); | ||
| } | ||
| String TxtViewerApp::onGetStrFontSpacing() { | ||
| return StringFormat("Font SPACE %d C(-) A(+)", fontSpacing); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be "D(-)".

No description provided.