-
Notifications
You must be signed in to change notification settings - Fork 34
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
Vue migration #30
base: master
Are you sure you want to change the base?
Vue migration #30
Conversation
Can you please check this, this and this Before:
After:
|
Thank you for your review. I will have a look at it! |
your issues should be fixed now. |
I hopefully fixed the slots issues and the 'alt-1-'s that I haven't replaced. There was also another bug in the image selector. Hopefully everything is working now! Sorry for such a buggy pull request. |
Hi @jannessm , first of all thank you very much for this incredible refactor!! I want to try it but with previous PR that I've accepted, now have been appear some conflicts. Can you resolve it please? I will try to review your PR and accept it asap. |
I added now the new style. other PRs I havent found except the readme update which should be no conflict when merging. Hope that everything works fine! |
Hi,
I just refactored your project to use vuejs. Now the code should be more cleaner and easier to extend by contribution.
Also I added buttons to add and remove animated lower thirds (up to 10)