Skip to content

Conversation

@vlastavesely
Copy link
Contributor

  • bug fix? yes

I just made Tracy more sexy :) There is issue opened that shows headings being affected by CSS property with !important set on. I have fixed this particular case (for headings and paragraphs).

Another problem occured on page with styled pseudoelements :before or :after. On my page, there are styled headings that are decorated by these pseudoelements. Sadly, Tracy rendered these as well and so its layout went broken. Now, after my fixes, any pseudoelements inside Tracy will be hidden by default except cases where it is need for it.

@vlastavesely vlastavesely changed the title bar.css, bluescreen.css: fixed leaks of page's default CSS bar.css, bluescreen.css: fixed leaks of page's default CSS into Tracy Jan 11, 2017
@adrianbj
Copy link
Contributor

On this note, I would really love if bar.css 's

#tracy-debug * {

could be changed to:

#tracy-debug-bar * {

I make this change every time there is an update so I can include ACE editor in a custom panel. I have mentioned this before, but was hoping that if some css changes are about to be made that this could be reconsidered.

Thanks and sorry for hijacking!

@vlastavesely
Copy link
Contributor Author

Since #tracy-debug-bar is inside #tracy-debug - at least in Tracy that I have installed on this machine (another from that I have sent PR from) - it should be affected also by my change. Am I missing something? What is exactly your problem?

@adrianbj
Copy link
Contributor

Am I missing something? What is exactly your problem?

I just tested with your changes and I still need to make that change of mine which makes things much easier to override.

My issue is actually kind of the reverse to what you are addressing here - sorry for the confusion - I just wanted to get the CSS discussion going again.

More info here: #151

@adrianbj
Copy link
Contributor

Just thought I'd also post the additional CSS rules I am adding (along with that change from #tracy-debug to tracy-debug-bar

https://github.com/adrianbj/TracyDebugger/blob/865a669a3ddfc0e528413452235e8aefe50e332e/styles.css#L1-L208

The highlighted lines are the ones relevant to all Tracy installs. The rest are only relevant to some of my custom panels. This combination seems to work well, but perhaps your additions here would avoid the need for some of my added rules?

@vlastavesely
Copy link
Contributor Author

So basically, your problem is that Tracy's style is damaging something you've put inside the panel? I fixed issues occurring when outside page's CSS damages Tracy's design. I believe these are two different things and should be solved separately. Your problem is primarily in the third party library you are using and there it should be fixed. Maybe it can be fixed in Tracy but, as I said, it is different problem.

However, for test purpose, I've just tried to change #tracy-debug * to #tracy-debug-bar *. There are some differences after I did so. For example, headings h2 look a little bit different. I do not have time to solve it on my own. If it bothers you, you should prepare your own PR with solution :)

@adrianbj
Copy link
Contributor

So basically, your problem is that Tracy's style is damaging something you've put inside the panel? I fixed issues occurring when outside page's CSS damages Tracy's design.

Yes, I realize the difference. I was just trying to contribute to improving the CSS conflicts in both directions. Anyway, sorry for hijacking your PR.

@dg
Copy link
Member

dg commented Jan 13, 2017

Thanks!

@dg dg merged commit 79513a9 into nette:master Jan 13, 2017
@dg
Copy link
Member

dg commented Jan 13, 2017

I merged only :before and :after reset, for the !important part I am trying a different approach #241.

@mrtnzlml
Copy link

Sending kudos from zlml.cz... :)
screenshot from 2017-01-13 09 35 59
screenshot from 2017-01-13 09 42 12

#tracy-debug *:before,
#tracy-debug *:after {
display: none;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMHO all: unset; is better

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