Skip to content
This repository has been archived by the owner on Oct 26, 2019. It is now read-only.

Summer 2018 Code Tweaks #868

Open
wants to merge 40 commits into
base: 151
Choose a base branch
from
Open

Conversation

kryptot7
Copy link

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 (Wick "fakes" bold typefaces instead of retrieving actual bold font from Google Fonts #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 and others added 30 commits April 14, 2018 14:56
TODO: If bold or italic is enabled and you choose a font that doesn't support either one, the font will refuse to load and throw a console error. Fix this.
Also, "Font loading..." appears in alert box now. The alertbox is also more sophisticated.
TODO: Proper font weight/italic still doesn't show up in Player. Also, it's still impossible to choose alternative weights like Light or ExtraBold.
-I removed all effects, leaving only clean white shapes
-I tweaked the designs of a few SVGs
-I horizontally and vertically aligned the contents of each SVG
Now more consistent with others
Also added new info svg for the alert box
Also fixed bug where mousing over other menu bar options while one is open does not open those up. Using the menu bar is much more intuitive now.
Multiplication sign in inspector is buggy
@zachrispoli
Copy link
Member

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
Copy link

@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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants