Closed
Bug 75659
Opened 24 years ago
Closed 21 years ago
charoff returned without % sign.
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: desale, Assigned: sicking)
References
()
Details
(Keywords: testcase)
Attachments
(1 file)
|
1.10 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
BUILDS:
2001-04-10-04 [Win95]
2001-04-10-04 [Mac]
2001-04-10-14 {Linux]
STEPS TO REPRODUCE:
1] Please visit URL above.
2] Click button "Show me charoff"
EXPECTED RESULTS: alert should show message
"document.getElementsByTagName('COL')[1].chOff = 25%"
ACTUAL RESULTS: alert shows
message"document.getElementsByTagName('COL')[1].chOff = 25"
it shows 25 instead of 25%
Pasting code used in testcase at above URL.
HTML CODE START
<HTML>
<HEAD>
<TITLE></TITLE>
</HEAD>
<BODY>
COL align="char" charoff="25%"
<TABLE border="1">
<COLGROUP>
<COL><COL align="char" charoff="25%">
<THEAD>
<TR><TH>The decimal points <TH>should line up
<TBODY>
<TR><TD>Dogs and cats<TD>$1
<TR><TD>living together.<TD>$10.50
<TR><TD>Mass hysteria!<TD>$1,000.30
</TABLE>
<script>
<!--
function showchOff(){
var t=document.getElementsByTagName('COL')[1].chOff;
alert("document.getElementsByTagName('COL')[1].chOff = " + t);
}
//-->
</script>
<FORM NAME="secondform">
EXPECTED RESULTS:
alert should show message "document.getElementsByTagName('COL')[1].chOff =
25%"<br><br>
ACTUAL RESULTS:
alert shows message"document.getElementsByTagName('COL')[1].chOff = 25" <br>
it shows 25 instead of 25%<br><br>
<INPUT TYPE="button" NAME="but1" VALUE="Show me charoff" onclick="showchOff();">
</FORM>
</BODY>
</HTML>
HTML CODE END
|
Reporter | |
Comment 1•24 years ago
|
||
Again seems DOM-HTML. Reassigning.
Assignee: karnaze → jst
Component: HTMLTables → DOM HTML
QA Contact: amar → desale
|
|
Reporter | |
Comment 3•24 years ago
|
||
Updating QA contact to Shivakiran Tummala.
QA Contact: desale → stummala
|
||
Comment 5•21 years ago
|
||
This just looks like a problem with the charoff parsing -- we always parse it as
an integer. sicking, wanna fix this while you're in this code?
|
|
||
Comment 6•21 years ago
|
||
Top be precise, see
http://lxr.mozilla.org/seamonkey/source/content/html/content/src/nsHTMLTableRowElement.cpp#520
Priority: P5 → --
Target Milestone: Future → ---
|
Assignee | |
Comment 7•21 years ago
|
||
Assignee: general → bugmail
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•21 years ago
|
Attachment #142881 -
Flags: superreview?(jst)
Attachment #142881 -
Flags: review?(jst)
|
||
Comment 8•21 years ago
|
||
Comment on attachment 142881 [details] [diff] [review]
patch to fix
r+sr=jst
Attachment #142881 -
Flags: superreview?(jst)
Attachment #142881 -
Flags: superreview+
Attachment #142881 -
Flags: review?(jst)
Attachment #142881 -
Flags: review+
|
|
Assignee | |
Comment 9•21 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Component: DOM: HTML → DOM: Core & HTML
QA Contact: stummala → general
You need to log in
before you can comment on or make changes to this bug.
Description
•