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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct "bottom of the screen" example #188

Merged
merged 1 commit into from
Feb 25, 2021
Merged

Correct "bottom of the screen" example #188

merged 1 commit into from
Feb 25, 2021

Conversation

pyfisch
Copy link

@pyfisch pyfisch commented Dec 25, 2020

Due to the line break emitted by the print statement the text doesn't appear at the bottom of the screen but one line above. Using end='' ensures the text appears at the bottom of the terminal window.

This took me a rather long time to figure out. 馃槅

Due to the line break emitted by the print statement the text doesn't appear at the bottom of the screen but one line above. Using end='' ensures the text appears at the bottom of the terminal window.
@codecov
Copy link

codecov bot commented Dec 25, 2020

Codecov Report

Merging #188 (dae84c9) into master (b22d3aa) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #188   +/-   ##
=======================================
  Coverage   95.32%   95.32%           
=======================================
  Files           9        9           
  Lines        1006     1006           
  Branches      160      160           
=======================================
  Hits          959      959           
  Misses         44       44           
  Partials        3        3           

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update b22d3aa...dae84c9. Read the comment docs.

@jquast
Copy link
Owner

jquast commented Feb 24, 2021

Thanks, I think we should also update some docs at https://blessed.readthedocs.io/en/latest/location.html and spell out in the first paragraph, the usual need for end='' for print function, and from __future__ import print_function for python 2.

@jquast jquast merged commit 8d4c6fa into jquast:master Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants