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

Add print sentence utility function. #33

Closed
ragnorc opened this issue Mar 10, 2020 · 4 comments
Closed

Add print sentence utility function. #33

ragnorc opened this issue Mar 10, 2020 · 4 comments
Assignees
Projects

Comments

@ragnorc
Copy link
Contributor

@ragnorc ragnorc commented Mar 10, 2020

No description provided.

@ragnorc ragnorc created this issue from a note in Louis (In progress) Mar 10, 2020
@Canbenix

This comment has been minimized.

Copy link
Contributor

@Canbenix Canbenix commented Mar 24, 2020

The cells near the end of the row will not go to blank when printing the last part of the sentence.
Example:
Printing "chocolate muffins" on 7 cells ->
c h o c o l a
t e ' ' m u f f
i n s m u f f
The space character corresponds to a blank output.

@Canbenix

This comment has been minimized.

Copy link
Contributor

@Canbenix Canbenix commented Mar 25, 2020

image

This needs fixing too. Right now each individual capital letter is indicated (which I think is the correct way of doing that but I haven't checked), but also each individual digit. Instead it should be as in the screenshot (taken from https://www.pharmabraille.com/pharmaceutical-braille/the-braille-alphabet/).

@ragnorc

This comment has been minimized.

Copy link
Contributor Author

@ragnorc ragnorc commented Mar 25, 2020

Okay, will take care of that

@ragnorc

This comment has been minimized.

Copy link
Contributor Author

@ragnorc ragnorc commented Mar 28, 2020

Can be closed as #47 fixes it.

@ragnorc ragnorc closed this in #47 Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Louis
  
In progress
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.