Make online version work in old browsers. #385
Conversation
docs/css/content.css
Outdated
| } | ||
| #content #output { | ||
| overflow: auto; | ||
| max-height: 20em; | ||
| margin: 0; padding: .5em 1em; border: 2px solid silver; border-top: none; | ||
| background-color: #f0f0f0; | ||
| background-color: silver; |
There was a problem hiding this comment.
Why exchange #f0f0f0 and silver? :)
There was a problem hiding this comment.
I made the output colored, and it was easier to change the backgrounds so that yellow integers showed up than it was to figure out a different color for integers. I'm willing to turn colored output back off, or change the output colors if you'd prefer.
There was a problem hiding this comment.
No, I meant that in the other place you changed silver to #f0f0f0 :)
There was a problem hiding this comment.
Anyway, it just seemed odd. If it was intentional, don't mind my comment
There was a problem hiding this comment.
I would use #c0c0c0 for readability though.
There was a problem hiding this comment.
You mean #c0c0c0 instead of silver, right?
| depth: Infinity, | ||
| color: false, | ||
| maxArrayLength: Infinity, | ||
| maxStringLength: Infinity, | ||
| compact: false, | ||
| stylize: util.stylizeWithHTML, |
There was a problem hiding this comment.
This is the place where coloration was added.
There was a problem hiding this comment.
and now I need to file a bug with node-inspect-extracted to HTML-escape output in stylizeWithHTML.
|
Depends on hildjj/node-inspect-extracted#22 |
|
OK, I think we're there. @polkovnikov-ph double check, please? |
Fixes #371.