Summer 2018 Code Tweaks

[kryptot7]

https://forum.wickeditor.com/t/wick-fork-with-new-ui-other-improvements/769 I made several commits to my fork over the summer, but I unfortunately never got around to making the actual pull request. 😞 Better late than never, though! With the overhaul to Wick being previewed in just a few days, I'm not sure if accepting these changes would confuse users, but I wanted to make them accessible for you in case you wanted to implement any of these to the old Wick before the new one is stable. Enjoy! 😄

The default branch in my fork is called ‘all-improvements’ and it’s based on the upstream branch ‘151’.

Changes

• Dark theme that makes heavy use of CSS variables for easy customization
• Cleaned up and/or redrew all Tool .svgs, removing all shadow effects (which showed up as vague fuzziness due to the small size of the icons)
• Changed font from Anonymous Pro to Segoe UI (Windows), San Francisco/Helvetica Neue (Mac), or sans-serif. I preferred this for look and readability, but feel free to change it to whatever you want by tweaking --ui-font in editor.css.
• Smaller, space-saving timeline
• Timeline can now be resized
• Fixed some timeline bugs related to resizing
• Got rid of “Inspector” label to save space, as I felt it was redundant
• Input fields in inspector now have styling
• Mousing over other menus while one is open behaves as you’d expect
• Visual improvements for menu bar selections and right-click menu
• Added ellipses to various menu items (like “Export…”)
• Various capitalization fixes throughout UI
• Alert box now tells you when a web font is loading or when loading fails
• Fonts in inspector now load their bold/italic variant (#768)
• Tweens now have rainbow colors, just for fun! (this may have a small impact on performance, though)
• Used consistent colors for buttons and highlight colors
• Script editor API headings don’t clip on bottom anymore
• Got rid of color picker close button, it now applies new color on clickout
• Color picker is more compact
• Added some alert box showMessage() functionality

[zachrispoli]

Hey Landon!

Sorry to get back to you super late on this - wick development has been pretty hectic lately as @Luxapodular and I have been doing a lot of administrative work related to grant money and such.

So here's the current situation with the editor: After trying to rewrite the editor from scratch a few months ago, I actually had to scrap that rewrite and try again. The editor that I'm currently working on here is actually a second rewrite, where the idea is that the GUI will use React, everything else will be a separate library/project (you'll see that the drawing tools, the timeline, the script/tick system etc. are now their own projects)

SO, basically, the first rewrite I did was pretty much compatible with your changes here and I was planning on merging your changes into that one, but since the second rewrite is using React, the GUI is all completely new code written in jsx.

However - the good news is that the changes you made to the CSS are still usable with React, and the fixes you added to the google font system should also still work. And of course, the graphical changes you did in various places (tool icons and such) can still be used. As long as work on the React editor continues at the same pace, I should be able to merge these fixes in the next few weeks.

Anyway, sorry for all the difficulties with contributing to the editor, I'm still pretty new to maintaining big projects like this but I'm learning ~~!! 😬

[yordan-kanchelov]

@zrispo just as a recommendation it would be good to add some of that information in the README project and link the new repo there. I have few colleagues who thought the project is dead considering the issues count/response rate and the commit activity