developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 10 years ago

Closed 10 years ago

#1151 closed bug (fixed)

jForms: when (de)activating a choice control, subelements should be marked as disabled instead of readonly

Reported by: Julien Owned by: Julien
Priority: normal Milestone: Jelix 1.2RC1
Component: jelix:forms Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description


Attachments (1)

cross.png (655 bytes) - added by Julien 10 years ago.
put in /lib/jelix-www/design/jforms/cross.png

Download all attachments as: .zip

Change History (8)

comment:1 Changed 10 years ago by Julien

  • review set to review?
  • Status changed from new to assigned

comment:2 follow-up: Changed 10 years ago by laurentj

  • review changed from review? to review-

why? what is the problem ?

However, the patch is not enough. Builders should generate also disabled attribute.

comment:3 in reply to: ↑ 2 Changed 10 years ago by Julien

Replying to laurentj:

why? what is the problem ?

I just think that marking the non-usable fields as disabled is better as marking them as readonly (still applicable but not updatable). I think it gives more sense. Kind of "semantic" thing :)

However, the patch is not enough. Builders should generate also disabled attribute.

ok you're right, I should have modified the builders too... If you're ok with the fact that "disabled" is better than "readonly", I'll complete the patch.

thanks for your answer Laurent

comment:4 Changed 10 years ago by laurentj

I think there is a reason why I set readonly instead of disabled. But I don't remember....

comment:5 Changed 10 years ago by Julien

In current testapp (testapp/www/index.php?module=testapp&action=sampleform:newform) we can see that readonly attribute is not good as at the bottom of the form, the "Closed Status" menulist can still be changed.

New mq patch is coming (modified builders too)

comment:6 Changed 10 years ago by Julien

  • review changed from review- to review?

here it is: http://bitbucket.org/julieni/jelix-trunk-patches/changeset/97e060d32aa7

I also made other improvements, so the patch is quite big, but most the changes are related to the initial problem

Changed 10 years ago by Julien

put in /lib/jelix-www/design/jforms/cross.png

comment:7 Changed 10 years ago by laurentj

  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.