-
Notifications
You must be signed in to change notification settings - Fork 0
Add an editor #1
Comments
Still need to put back some style on button etc... and fix code. |
Hmm, I liked this version! Maybe adding information about that tool at the top? |
Looks good as an initial version! We can improve it with more modern styles (maybe a lot of JS/CSS works needed) I can also work for it and send you a PR soon. |
Thank you for the review. I'll continue fixing errors. @leandrodamascena Do you know about how the editor can handle the error I found nothing in the doc? @mingrammer I put the default theme maybe there's certainly one with better render, depends on what you plan to do, here the list: And I saw that the editor can handle redo and undo actions... https://ace.c9.io/demo/toolbar.html I'll try this one. There's a lot to do! |
I'm not sure about the error handler in this editor.. But we might use try blocks in flask to check this information and returns a human readable error.. I'll check one thing in diagrams code, thinking about get error from there |
I think we should show the thrown stack traces as it is with pop-up like interface on the editor panel. I used toastr library for such this purpose before. |
Great idea @mingrammer. I already used toastr and it's really good to show messages. |
Hi guys. Leaving feedback, I had some difficult days here. But now I am back and I can work on it. |
I created a team and took the diagams-web before somebody else take it :) I sent you invitations with team we will have discussion tools it will make more easy to manage as I'm lost with all the issues... You also become owner so it will make also more easy to manage. |
The text was updated successfully, but these errors were encountered: