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

Using attribute such as ngModel, required, disabled, maxlength... ? #31

Open
Seiifer opened this issue Apr 4, 2019 · 4 comments
Open
Assignees
Labels

Comments

@Seiifer
Copy link

Seiifer commented Apr 4, 2019

Is there a simple way to use those attributes on the angular wrapper ?
Shouldn't it be added to the component in order to be able to use them when it's calling jodit ?

Using (onChange) instead of [(ngModel)] is very heavy and I can feel the performances drop.

For now I'm using a hidden input with the proper maxlength and required attribute and a [(ngModel)] updated on the (onChange) event from the editor input.
But it's getting slow, and I couldn't find a way to disable the jodit editor yet.

There must be a better solution.
Any advice ?

@katan
Copy link

katan commented Apr 25, 2019

I don't understand why you require to implement those attributes inside Jodit.
You must be implement it on your component controller or on a class model.

In my case, I use Reactive Forms instead ngModel and those attributes (like required, disabled ... attributes) are implement there and all runs ok.

For disable/enable the Jodit editor see the documentation:
https://xdsoft.net/jodit/doc/options/disabled/

@suhaskurade8
Copy link

I'm facing the same problem. Maxlength is not working in my case

@MrElectroNick
Copy link
Collaborator

Hi @Seiifer @suhaskurade8 - [(ngModel)] is now supported in latest versions of jodit-angular
I'll take a look if it is possible to support [maxLength] als and if so will close this ticket with next update
Cheers!

@MrElectroNick MrElectroNick self-assigned this Mar 20, 2021
@kumarbrajesh4001
Copy link

I want set required field for validation , but not working

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants