6.0.0-alpha12
6/12/25

[#3705] Show/Hide Javascript Malfunction
Summary Show/Hide Javascript Malfunction
Queue Kronolith
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners
Requester nchanda (at) tachometry (dot) com
Created 03/28/2006 (7016 days ago)
Due
Updated 03/28/2006 (7016 days ago)
Assigned
Resolved 03/28/2006 (7016 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
03/28/2006 10:11:35 PM Jan Schneider Comment #3
State ⇒ Resolved
Reply to this comment
Committed, thanks.
03/28/2006 06:54:25 PM nchanda (at) tachometry (dot) com Comment #2 Reply to this comment
This patch has implications on the implementation of the "Display 
Object" additions throughout the groupware applications (see 
https://dev.horde.org/horde/whups/ticket/?id=3424).



As the other applications are brought in-line with the Display 
Calendars area, the checkbox event should be consistent.
03/28/2006 06:44:29 PM nchanda (at) tachometry (dot) com Comment #1
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Show/Hide Javascript Malfunction
Queue ⇒ Kronolith
New Attachment: cal_showhide_checkbox.patch.txt Download
Reply to this comment
The submitted patch modifies the show / hide checkboxes to use the 
onClick event trigger instead of the onChange event.  Implementation 
of this patch creates consistant cross-browser functionality.



The showHide javascript function in calendars.inc is called by the 
onChange event handler when the checkbox is clicked.  Firefox works 
well, submitting the form upon mouse up.  IE does not register the 
event until focus is removed from the checkbox by clicking (or 
tabbing) to another location on the page.



W3C recommends that the onChange event should only be triggered when 
the element looses focus, meaning that IE has actually implemented 
this correctly.  While not as descriptive, the onCLick event should be 
used instead.



Here is the W3C statement on the onChange event:

http://www.w3.org/TR/REC-html40/interact/scripts.html#adef-onchange


Saved Queries