Closed Bug 134716 Opened 22 years ago Closed 22 years ago

the javascript onClick attribute fails if used many times in the <TD> tag

Categories

(Core :: Layout: Tables, defect, P3)

Product:
Core
Component:
Layout: Tables
Platform:
x86
All
Type:
defect

Tracking

()

Status:
RESOLVED WORKSFORME
Milestone:
Future

People

(Reporter: fulco, Assigned: karnaze)

References

(
URL
)

Details

(Keywords: testcase)

Attachments

(2 files)

testcase
14.01 KB, text/html
Details
Testcase from reporter (bug 134822)
3.78 KB, text/html
Details 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.9) Gecko/20020310
BuildID:    2002031008

Only the first nine data elements from the agenda below react on onClick.If you
view this example in IE or Opera it works fine.  

Example: This page displays an agenda, if you click on an empty line it should
react with a new appointment. Only the first nine rows do what you expect them
to do.



Reproducible: Always
Steps to Reproduce:
1.load the example
2.click on an empty appointment row
3.a new page should be loaded (it is not included in the example)

Actual Results:  if you click on the first nine appointment rows everything
works fine.
if you clock on the other rows nothing happens.

Expected Results:  Like Opera and IE react to the click.

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN">
<html>
<head>

<link REL="stylesheet" TYPE="text/css" HREF="./styles/maresth.css">
<meta HTTP-EQUIV="CONTENT-TYPE" CONTENT="text/html; charset=iso-8859-1">
<meta NAME="AUTHOR" CONTENT="Fulco Bohle">
</head>

<script LANGUAGE="JavaScript">

function toon_volgende_maand( jaar, maand)
	{
	window.location = "home.php?operation=next_month";
	}

function toon_voorgaande_maand( jaar, maand)
	{
	window.location = "home.php?operation=previous_month";
	}

function toon_dag( jaar, maand, dag)
	{
	window.location = "home.php?operation=show_day&maand=" + maand + "&jaar=" + jaar
+ "&dag=" + dag;
	}

function wijzigen_afspraak( nummer)
	{
	window.location = "home.php?operation=edit_appointment&afspraaknummer="+nummer
	}

function verwijderen_afspraak( nummer)
	{
	window.location = "home.php?operation=delete_appointment&afspraaknummer="+nummer
	}

function nieuwe_afspraak( datum, tijd)
	{
	window.location =
"home.php?operation=new_appointment&afspraaknummer=0&begintijd="+tijd+"&datum="+datum
	}

</SCRIPT>

<body>

<table WIDTH="100%" BORDER="1">

<td ALIGN="CENTER">AGENDA</td>
<td ALIGN="CENTER">TAKEN</td>
<td ALIGN="CENTER">KLANTEN</td>
<td ALIGN="CENTER">KENNISBANK</td>
<td ALIGN="CENTER">INKOOP</td>
<td ALIGN="CENTER">AFMELDEN</td>
<tr>
</table>
<table WIDTH="100%" BORDER="1">
<td ROWSPAN="2"><table CLASS="calendar" BORDER="0" RULES="ALL" WIDTH="100%">
<tr><th COLSPAN="2">tijd</th><th WIDTH="90%">afspraak</th></tr>

<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">8</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '08:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '08:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">9</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '09:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '09:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">10</td><td
CLASS="calendar_minutes">00</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '10:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '10:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">11</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '11:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '11:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">12</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '12:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '12:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">13</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '13:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '13:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">14</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '14:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '14:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">15</td><td
CLASS="calendar_minutes">00</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '15:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '15:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">16</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '16:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '16:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">17</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '17:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '17:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">18</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '18:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '18:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">19</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '19:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '19:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">20</td><td
CLASS="calendar_minutes">00</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '20:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '20:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">21</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '21:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '21:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">22</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '22:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>

<td onClick="nieuwe_afspraak( '2002-4-2', '22:30');"></td></tr>
<tr><td CLASS="calendar_hours" VALIGN="TOP" ROWSPAN="2">23</td><td
CLASS="calendar_minutes">00</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '23:00');"></td></tr>
<tr><td CLASS="calendar_minutes">30</td>
<td onClick="nieuwe_afspraak( '2002-4-2', '23:30');"></td></tr>
</table>
</td>
<td WIDTH="280" HEIGHT="160">
<table CLASS="calendar" BORDER="0" RULES="ALL">
<td CLASS="calendar" COLSPAN="2"
onClick="toon_voorgaande_maand(2002,4);">vorige</td>
<td CLASS="calendar_month" COLSPAN="3" >april 2002</td>
<td CLASS="calendar" COLSPAN="2"
onClick="toon_volgende_maand(2002,4);">volgenda</td>

<tr><td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >zo</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >ma</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >di</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >wo</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >do</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >vr</td>
<td CLASS="calendar_weekdays" NOWRAP ALIGN="CENTER" VALIGN="MIDDLE" >za</td>
<tr><td CLASS="calendar"></td>
<td WIDTH="40" HEIGHT="20"  CLASS="calendar_weekdays"
onClick="toon_dag(2002,4,1);">1</td>

<td CLASS="calendar_currentday" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,2);">2</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,3);">3</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,4);">4</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,5);">5</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,6);">6</td>
</tr>
<tr><td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,7);">7</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,8);">8</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,9);">9</td>

<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,10);">10</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,11);">11</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,12);">12</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,13);">13</td>
</tr>
<tr><td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,14);">14</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,15);">15</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,16);">16</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,17);">17</td>

<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,18);">18</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,19);">19</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,20);">20</td>
</tr>
<tr><td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,21);">21</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,22);">22</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,23);">23</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,24);">24</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,25);">25</td>

<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,26);">26</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,27);">27</td>
</tr>
<tr><td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,28);">28</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,29);">29</td>
<td CLASS="calendar_weekdays" WIDTH="40" HEIGHT="20"
onClick="toon_dag(2002,4,30);">30</td>
<td CLASS="calendar"></td>
<td CLASS="calendar"></td>
<td CLASS="calendar"></td>
<td CLASS="calendar"></td>
</tr>
<td CLASS="calendar" HEIGHT="20"></td>

<td CLASS="calendar" HEIGHT="20"></td>
<td CLASS="calendar" HEIGHT="20"></td>
<td CLASS="calendar" HEIGHT="20"></td>
<td CLASS="calendar" HEIGHT="20"></td>
<td CLASS="calendar" HEIGHT="20"></td>
<td CLASS="calendar" HEIGHT="20"></td>
</tr>
</table>
</td><tr><td VALIGN="TOP"><table BORDER="0" FRAME="(void)">
<tr><th BGCOLOR="#D0D0A0">project</th><th BGCOLOR="#D0D0A0">onderwerp</th></tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(22);">Texass</td>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(22);">Portefeuille Aegon Leven van de
Ridder</td>

</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(23);">Texass</td>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(23);">Commerciele korting</td>
</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(24);">Athene</td>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(24);">ZVP aanpassing naverrekening</td>
</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(25);">Texass</td>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(25);">Windows XP</td>

</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(28);">Texass</td>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(28);">Schade dossiers volmacht</td>
</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(20);">Texass</td>
<td BGCOLOR="#FFF0C0" onClick="ModTaak(20);">Afwijkend incasso</td>
</tr>
<tr ALIGN=LEFT VALIGN=TOP>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(27);">Athene</td>
<td BGCOLOR="#F0E0B0" onClick="ModTaak(27);">Selectie Janneke prolongatie HBR
Branche</td>

</tr>
</table>
</td><tr></table>
<hr>
username=[root]<br>
current_state=[agenda]<br>
state_stack[0]=[login]<br>
agenda_jaar=[2002]<br>
agenda_maand=[4]<br>
agenda_dag=[2]<br>

<br>posted variables:<hr>
<hr>
username=[root]<br>
operation=[]<br>
Your session ID is eed16c93cb1ae832421b4cbca80658b2<br>
<hr>select * from agenda where datum = "2002-4-2";<hr>

</body>
</html>
Attached file testcase — 
please make an attachment next time.
the testcase seems to want to use a script to create the "appointments", which
doesn't work here.  I don't see what you see, but that could be because new
appointments don't work.  could you construct a testcase that would work here?  

Probably if you create the appointments to the point that it stops working and
then save that page and see if that works.

You might also try out a recent nightly build.
*** Bug 134822 has been marked as a duplicate of this bug. ***
Blocks: 134822
Confirming bug with Mozilla trunk binary 20020330xx on WinNT. If you load
the testcase in attachment #4 [details] [diff] [review], you get alertboxes for the first few rows,
but not the later ones. No errors in the JavaScript Console. 

Browser, not engine. Reassigning to HTML Tables for parity with the
other bug. 

Note: the size of my Mozilla window affects this. If I have Mozilla
maximized, the first row that fails for me is the 11:00 row. But if
I have Mozilla shrunk down to a small size, the first row that fails
is the 10:00 row!
Assignee: rogerl → karnaze
Status: UNCONFIRMED → NEW
Component: JavaScript Engine → HTMLTables
Ever confirmed: true
OS: Linux → All
QA Contact: pschwartau → amar
No longer blocks: 134822
*** Bug 134822 has been marked as a duplicate of this bug. ***
Note that the height parameter in the example affects the area that does not respond: see the line 
that contains

<td WIDTH="280" HEIGHT="160">if you click

and modify the height to lets 
say 260

Fulco
Keywords: testcase
Priority: -- → P3
testcae worksforme with linux build 20020505 (trunk)
Bug resolved on Linux build 2002050910 
Bug resolved on Windows build 2002050908

I am a happy man.

Fulco
Target Milestone: --- → Future
marking as wfm based on reporters comment
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: