intercooler-js icon indicating copy to clipboard operation
intercooler-js copied to clipboard

Create ic-disable-inputs attribute

Open zodman opened this issue 5 years ago • 11 comments

On a request we closestAttrValue('ic-disable-inputs') of the element submitting. If it is true, we search for all inputs below that element and disable them.

Then you could put ic-disable-inputs on a form and have it disable all input elements below, put it on a single input, etc.

zodman avatar May 02 '19 01:05 zodman

some can check this?¿

zodman avatar Jun 04 '19 06:06 zodman

@chg20 can check it plz!

zodman avatar Jun 04 '19 06:06 zodman

I'm a little worried that this would cause a big change in behavior. Maybe it would be OK if the element type were verified?

1cg avatar Jun 05 '19 17:06 1cg

@chg20 done!

zodman avatar Jun 05 '19 23:06 zodman

@chg20 can you take a look! thanks in advance.

zodman avatar Jun 13 '19 03:06 zodman

ping!

zodman avatar Jul 23 '19 21:07 zodman

Can I ask that this be reworked and expanded a bit?

I think the right behavior is to add a new attribute: ic-disable-inputs="true"

On a request we closestAttrValue('ic-disable-inputs') of the element submitting. If it is true, we search for all inputs below that element and disable them.

Then you could put ic-disable-inputs on a form and have it disable all input elements below, put it on a single input, etc.

Does this sound reasonable?

I also would like a test for this if possible.

1cg avatar Aug 09 '19 17:08 1cg

for the hack October https://hacktoberfest.digitalocean.com/ i will working with this!

zodman avatar Oct 05 '19 17:10 zodman

awesome 👍

On Oct 5 2019, at 10:44 am, Andres Vargas [email protected] wrote:

for the hack October https://hacktoberfest.digitalocean.com/ i will working with this!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub (https://github.com/intercoolerjs/intercooler-js/pull/284?email_source=notifications&email_token=AADSRP47UZKUI5XRCIWNT2LQNDHBVA5CNFSM4HJ24BT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEANX2DQ#issuecomment-538672398), or mute the thread (https://github.com/notifications/unsubscribe-auth/AADSRP7O3MK7PPJSWAODVWLQNDHBVANCNFSM4HJ24BTQ).

1cg avatar Oct 08 '19 16:10 1cg

@zodman I'm planning on doing a realase at the end of hacktober, let me know if you need any help or encouragement getting this done. I think it is a good feature!

1cg avatar Oct 11 '19 20:10 1cg

OK! I will integrate this, but it needs docs!

1cg avatar May 12 '20 19:05 1cg