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.

Sign up for GitHub

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

Jump to bottom

It allow edit READONLY inputs #63

Closed
robsontenorio opened this issue Oct 13, 2023 · 1 comment
Closed

It allow edit READONLY inputs #63

robsontenorio opened this issue Oct 13, 2023 · 1 comment
Assignees
@codermarcos
Labels
bug A confirmed bug

Comments

@robsontenorio
Copy link

It is supposed to not allow edit on <input readonly />. But, em setMask() runs the field can be edited.
@codermarcos codermarcos self-assigned this Oct 18, 2023
@codermarcos codermarcos added the bug A confirmed bug label Oct 18, 2023
@codermarcos codermarcos moved this to ToDo in Tasks Oct 18, 2023
codermarcos added a commit that referenced this issue Jan 28, 2024
@codermarcos
🐛 fix readonly inputs
f066669 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   cypress/e2e/v4.x.x/set-mask.cy.ts
	modified:   cypress/file/index.html
	modified:   src/set-mask.ts
	modified:   tests/set-mask.test.ts
codermarcos added a commit that referenced this issue Jan 28, 2024
@codermarcos
📝 fix doc error issue example
e049ac3 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   CONTRIBUTING.md
@codermarcos codermarcos mentioned this issue Jan 28, 2024
Merged
1 task
codermarcos added a commit that referenced this issue Jan 28, 2024
@codermarcos
🔀 Merge branch 'main' of github.com:codermarcos/simple-mask-money int…
73d707d 
…o fix/#63/readonly-inputs

Conflicts:
	src/set-mask.ts

On branch fix/#63/readonly-inputs
All conflicts fixed but you are still merging.

Changes to be committed:
	modified:   package-lock.json
	modified:   package.json
	modified:   src/get-base-configuration.ts
	modified:   src/set-mask.ts
	modified:   src/types.ts
	modified:   tests/get-base-configuration.test.ts
codermarcos added a commit that referenced this issue Jan 28, 2024
@codermarcos
🔀 Merge pull request #68 from codermarcos/fix/#63/readonly-inputs
030573f 
🐛 Fix readonly inputs
@codermarcos codermarcos moved this from ToDo to Doing in Tasks Jan 28, 2024
@codermarcos codermarcos moved this from Doing to Done in Tasks Jan 28, 2024
@codermarcos
Copy link
Owner

Thank you for your issue it was fixed with version 4.1.1.

codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🐛 fix readonly inputs
c3d8c32 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   cypress/e2e/v4.x.x/set-mask.cy.ts
	modified:   cypress/file/index.html
	modified:   src/set-mask.ts
	modified:   tests/set-mask.test.ts
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
📝 fix doc error issue example
2cb8c18 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   CONTRIBUTING.md
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🔀 Merge branch 'main' of github.com:codermarcos/simple-mask-money int…
bda0fcb 
…o fix/#63/readonly-inputs

Conflicts:
	src/set-mask.ts

On branch fix/#63/readonly-inputs
All conflicts fixed but you are still merging.

Changes to be committed:
	modified:   package-lock.json
	modified:   package.json
	modified:   src/get-base-configuration.ts
	modified:   src/set-mask.ts
	modified:   src/types.ts
	modified:   tests/get-base-configuration.test.ts
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🔀 Merge pull request #68 from codermarcos/fix/#63/readonly-inputs
32cd9bb 
🐛 Fix readonly inputs
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🐛 fix readonly inputs
d7b242f 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   cypress/e2e/v4.x.x/set-mask.cy.ts
	modified:   cypress/file/index.html
	modified:   src/set-mask.ts
	modified:   tests/set-mask.test.ts
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
📝 fix doc error issue example
4e91deb 
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   CONTRIBUTING.md
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🔀 Merge branch 'main' of github.com:codermarcos/simple-mask-money int…
34375e8 
…o fix/#63/readonly-inputs

Conflicts:
	src/set-mask.ts

On branch fix/#63/readonly-inputs
All conflicts fixed but you are still merging.

Changes to be committed:
	modified:   package-lock.json
	modified:   package.json
	modified:   src/get-base-configuration.ts
	modified:   src/set-mask.ts
	modified:   src/types.ts
	modified:   tests/get-base-configuration.test.ts
codermarcos added a commit that referenced this issue Aug 23, 2024
@codermarcos
🔀 Merge pull request #68 from codermarcos/fix/#63/readonly-inputs
5a43e61 
🐛 Fix readonly inputs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codermarcos codermarcos

Labels
bug A confirmed bug
Projects
Tasks
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@robsontenorio @codermarcos