-
Notifications
You must be signed in to change notification settings - Fork 7
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
No distinction between normal messages and hoon eval #23
Comments
Are we talking about moving the eval results outside of hover and putting them directly in the messages list? As long as the height is known, it shouldn't be a problem. Maybe a show/hide toggle would be better, as some of the results are quite long. I suppose the main problem here is that hoon eval messages are not styled differently enough. Maybe add a background color #f7f7f7 like the way markdown renders code blocks. |
On hover (I think) they appear inside of a fixed-height div that has |
I like the "use the same code class as tree" idea. They do appear monospace, but that's still visually confusing for e.g. On Thursday, 14 July 2016, Galen Wolfe-Pauly [email protected]
|
This is exacerbated by the static heights completely hiding the eval results, which do have code-block styling.
The text was updated successfully, but these errors were encountered: