Skip to content

Debugger Connection Panel explain block - non breakable spaces - #134

Closed
smartdev-cz wants to merge 3 commits into
nette:masterfrom
smartdev-cz:master
Closed

Debugger Connection Panel explain block - non breakable spaces#134
smartdev-cz wants to merge 3 commits into
nette:masterfrom
smartdev-cz:master

Conversation

@smartdev-cz

Copy link
Copy Markdown

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

current
formatted

@dg

dg commented May 19, 2016

Copy link
Copy Markdown
Member

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

@smartdev-cz

Copy link
Copy Markdown
Author

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

@dg

dg commented May 19, 2016

Copy link
Copy Markdown
Member

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

@JanTvrdik

Copy link
Copy Markdown
Contributor

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

@smartdev-cz

Copy link
Copy Markdown
Author

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

Copy link
Copy Markdown

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.

@dg
dg force-pushed the master branch 2 times, most recently from 59a6661 to b660054 Compare May 30, 2016 14:28
@dg

dg commented Jun 2, 2016

Copy link
Copy Markdown
Member

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

@dg
dg force-pushed the master branch 2 times, most recently from 5c87d56 to d9fd67a Compare June 27, 2016 23:11
@dg
dg force-pushed the master branch 13 times, most recently from 44a230d to 829b936 Compare October 18, 2016 13:01
@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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants