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

fix: htmlSet error in modal #127

Closed
wants to merge 1 commit into from
Closed

fix: htmlSet error in modal #127

wants to merge 1 commit into from

Conversation

akki-jat
Copy link

For more details: #107
Maybe also related to #87 but didn't check with v-if

Also there is one more PR open maybe for the same thing I am trying to do (but I did go with small change), also look at that: #98.

My error case

Error was coming for recreating editor because in initEvents it keeps reference of old VueComponent and that even again calls again the new instance is created but it gets _editor null because we destroyed it.

Also it can create memory leak problems because it was keeping VueComponent in init event callback.

@akki-jat
Copy link
Author

@stefanneculai can you take a look at this?

jyyan added a commit to jyyan/vue-froala-wysiwyg that referenced this pull request Jan 15, 2020
@gabaum10
Copy link

Does anyone know who's maintaining this? We'd like to get this merged/resolved. I'm tired of fielding "there is this mystery error in the console, fix it!" requests.

@rcheung9
Copy link

rcheung9 commented Feb 7, 2020

Is this going to be merged?

@akki-jat @gabaum10 @stefanneculai

@akki-jat
Copy link
Author

akki-jat commented Feb 8, 2020

I have also raised support ticket for the same.

@akki-jat
Copy link
Author

@kapil2704 can you look at this.

@paresh-accolite paresh-accolite mentioned this pull request Sep 25, 2020
@akki-jat
Copy link
Author

Closing this MR as changes are already merged with #157

@akki-jat akki-jat closed this Nov 16, 2020
@akki-jat akki-jat deleted the Fix_htmlSet_Error branch November 16, 2020 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants