-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
replacing codeflask with code-input #2139
base: master
Are you sure you want to change the base?
Conversation
I have some css related issue in this PR. Let me try debugging and closing this issue before this weekends. |
30289d3
to
80e6dd1
Compare
Could this PR be reviewed? |
Hi @Bowrna. There seems to be several issues with this. The HTML editor is completely broken. This is what I'm getting: Referring |
@knadh Let me fix this and update here. I missed this. |
this.initHTMLEditor(this.$props.value || ''); | ||
}, | ||
|
||
watch: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the below code being used only in markdownEditor and not in HTMLeditor?
watch: {
value(newVal) {
if (newVal !== this.data) {
this.data = newVal;
}
},
},
80e6dd1
to
24d6710
Compare
related: #2072
This is work in progress to replace codeflask with code-input for showing syntax highlighted text/code editor inside campaign content.