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

Debugger Connection Panel explain block - non breakable spaces #134

Closed
wants to merge 3 commits into from

Conversation

@WyskyNet
Copy link

WyskyNet commented May 19, 2016

Tiny styling fix for explain block in database connection panel...

current
formatted

@dg

This comment has been minimized.

Copy link
Member

dg commented May 19, 2016

What about add style #tracy-debug .nette-DbConnectionPanel-explain td { whitespace: pre }?

@WyskyNet

This comment has been minimized.

Copy link
Author

WyskyNet commented May 19, 2016

Hi David,
white-space: pre is usable too, but maybe not supported by older browsers...

@dg

This comment has been minimized.

Copy link
Member

dg commented May 19, 2016

Tracy is for IE10+, so it is not problem.

@JanTvrdik

This comment has been minimized.

Copy link
Contributor

JanTvrdik commented May 19, 2016

white-space: pre or white-space: pre-wrap?

@WyskyNet

This comment has been minimized.

Copy link
Author

WyskyNet commented May 23, 2016

Hi,
I am not sure about pre / pre-wrap, because i don't use explain regularly, but if I must select one of them, my choice will be pre (one line for one explain statement) :)

@repli2dev

This comment has been minimized.

Copy link

repli2dev commented May 23, 2016

pre seems preferable to me as the pre-wrap would wrap the long text into multiple lines where the following lines wouldn't be indented at all destroying the benefits of the indention at all.

WyskyNet added 2 commits May 23, 2016
@dg dg force-pushed the nette:master branch 2 times, most recently from 59a6661 to b660054 May 30, 2016
@dg

This comment has been minimized.

Copy link
Member

dg commented Jun 2, 2016

We can try pre and in case of troubles replace it with pre-wrap

@dg dg force-pushed the nette:master branch 2 times, most recently from 5c87d56 to d9fd67a Jun 27, 2016
@dg dg force-pushed the nette:master branch 13 times, most recently from 44a230d to 829b936 Oct 18, 2016
@dg dg closed this in 9efb63c Oct 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.