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

Whitespace characters are not printed correctly in the GUI console. #758

Open
Stinkfist0 opened this issue Feb 18, 2014 · 3 comments
Open
Labels

Comments

@Stinkfist0
Copy link
Contributor

f.ex. printing

                     foo

appears as

foo

in the GUI console. Also f.ex. newlines are omitted.

@Stinkfist0 Stinkfist0 added the Bug label Feb 18, 2014
@jonnenauha
Copy link
Member

Its probably calling QString::Simplified which removes all extra whitespace/newlines from the string. Why is it important to get spacing into UI console. The indents will show fine in cmd propt which is what you should be using for this kind of stuff. I assume indenting is only important for getting somwhat structured debug prints.

@Stinkfist0
Copy link
Contributor Author

For indentations it's not that critical but f.ex. an XML becomes a horrible one-liner to read or reuse. Also copy-pasting from the native console is much more annoying compared to the QTextEdit.

@jonnenauha
Copy link
Member

As a matter of fact we should just make the font monospace. Otherwise even if the whitespace would be there, it would look horrible. Good old Courier New 10-12px +1 :D Should be a trivial stylesheet change away.

Stinkfist0 referenced this issue in Adminotech/tundra Feb 26, 2014
…e neutral greyish. Tweaks stylesheets. Make opacity only affect the background, text will remain sharp and readable. Micro optimize DecorateString as its hit potentially a ton. Make config configurable buffer size, default in 5k lines which is plenty for normal use (fixed crashing client if something spams loggingm eg. scripts). Fix whitespace to remain in the ui as it was intented to be printed, this changes each block to a pre tag (in DecorateString).
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

2 participants