Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/263#issuecomment-136287425
Just tested it out a bit.
I'm not a big fan of the hover effect on the error (it gets a bit
transparent)
I also think the message time should be reduced to something like 6000ms,
currently its too long.
I'm not getting the console.log anymore when the error is showing.
(Need to check it out, because we can't have those errors not showing in
console)
I think it could also be nice not to include any error call in this PR
(just the service) and add error calls in another PR.
Because in current Zeppelin state, there is a lot of problems making the
error spam a lot, for example:
* I get the angular error everytime I hide/show the editor code.
* I get the angular error 3 times when running the paragraph (because bind
is called 3 times)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
