Skip to content
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

Refactoring and Vuejs #314

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

modos189
Copy link
Contributor

Reasons are described in #200

Simple connection Vue does not break anything, but the point is to rewrite interface on modern tools and create a convenient API to work with it.

Therefore, to support plugins that use old DOM intervention method, we need code that detects changes and translates it to new API.

For now, I just divided interface into base blocks, which I put into separate .vue files. For those who didn't work with Vuejs, these files are a collection of html, js and css for a single block.

I also rewrote the toolbox and created an API to add links to the toolbox (see toolbox.vue file):
vm.toolbox.add_item({id: 'mybtn', value: 'My button'})

To support the old plugins, I wrote a translator that tracks the appearance of new items in a hidden old toolbox and calls the new API (see sidebar.vue).

@johnd0e johnd0e added development general development issue core labels Feb 17, 2020
@johnd0e johnd0e mentioned this pull request Mar 3, 2020
@modos189 modos189 force-pushed the master branch 3 times, most recently from a34b800 to 7b9edd5 Compare December 12, 2020 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core development general development issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants