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

Add disabled state to RichContenteditable and clean not-working v-model #1894

Merged
merged 1 commit into from
May 4, 2021

Conversation

skjnldsv
Copy link
Contributor

@skjnldsv skjnldsv commented Apr 29, 2021

In order: normal, contenteditable=false, disabled=true

image

@skjnldsv skjnldsv added enhancement New feature or request 3. to review Waiting for reviews feature: rich-contenteditable Related to the rich-contenteditable components labels Apr 29, 2021
@skjnldsv skjnldsv added this to the 4.0.0 milestone Apr 29, 2021
@skjnldsv skjnldsv self-assigned this Apr 29, 2021
@nickvergessen

This comment has been minimized.

@skjnldsv

This comment has been minimized.

Base automatically changed from enh/switch to master April 29, 2021 10:35
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@@ -31,7 +31,7 @@ This component displays contenteditable div with automated @ autocompletion [at]
<template>
<div>
<RichContenteditable
v-model="message"
:value.sync="message"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing the not-working v-model

@@ -352,7 +376,6 @@ export default {
updateValue(htmlOrText) {
const text = this.parseContent(htmlOrText)
this.localValue = text
this.$emit('input', text)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixing the not-working v-model

@skjnldsv skjnldsv changed the title Add disabled state to RichContenteditable Add disabled state to RichContenteditable and rm not-working v-model Apr 29, 2021
@skjnldsv skjnldsv changed the title Add disabled state to RichContenteditable and rm not-working v-model Add disabled state to RichContenteditable and clean not-working v-model Apr 29, 2021
@skjnldsv
Copy link
Contributor Author

skjnldsv commented May 4, 2021

Ping!

@juliusknorr juliusknorr merged commit 1778824 into master May 4, 2021
@juliusknorr juliusknorr deleted the fix/contenteditable branch May 4, 2021 14:33
@skjnldsv skjnldsv mentioned this pull request Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request feature: rich-contenteditable Related to the rich-contenteditable components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants