children["id"].innerText - does not work with netscape but with explorer

  • Thread starter Julia Peterwitz
  • Start date
J

Julia Peterwitz

I have a function that works with explorer but not with netscape.
The problem is the function at line 5.


1 fSetSelectedDay(myElement){
2 /*
3 ...
4 */
5 var elementText = eval(myElement.children["calDateText"].innerText);
6 /*
7 ...
8 */
9 }
10 <!-- ... -->
11 <td id=calCell onclick='fSetSelectedDay(this)'>
12 <font id='calDateText' onclick='fSetSelectedDay(this)'>
13 <script> myMonth[w][d] </script>
14 </font>
15 </td>
16 <!-- ... -->


I need some support for this.
Thank you very much.
 
R

Randy Webb

Julia said:
I have a function that works with explorer but not with netscape.
The problem is the function at line 5.


1 fSetSelectedDay(myElement){
2 /*
3 ...
4 */
5 var elementText = eval(myElement.children["calDateText"].innerText);

..innerText is IE only, and as such is not going to work in NS. use
innerHTML instead.

eval is not needed there:

var elementText =
document.getElementById(myElement).children('calDateText').innerHTML;

the [ ] are also an IE-ism.
 
M

Michael Winter

Julia said:
I have a function that works with explorer but not with netscape.
The problem is the function at line 5.


1 fSetSelectedDay(myElement){
2 /*
3 ...
4 */
5 var elementText = eval(myElement.children["calDateText"].innerText);

.innerText is IE only, and as such is not going to work in NS. use
innerHTML instead.

eval is not needed there:

Agreed, however...
var elementText =
document.getElementById(myElement).children('calDateText').innerHTML;

....there are two problems with this line:

1) If you look at the OP's HTML, you'll see that myElement is a
reference to the element, not an id.
2) The children property is
a) a collection, so the brackets were correct,
b) also a Microsoft-ism.

Perhaps

var elementText = myElement.childNodes[ 'calDateText' ].innerHTML;

would be better (preferably with feature detection)?

Mike
 
J

julia peterwitz

myElement.childNodes[ 'calDateText' ].innerHTML;
is also not working
(it neither works in explorer nor in netscape)

any other proposals for netscape??

solution for explorer again:
myElement.children["calDateText"].innerText;
 
M

Michael Winter

myElement.childNodes[ 'calDateText' ].innerHTML;
is also not working
(it neither works in explorer nor in netscape)

any other proposals for netscape??

Oops. I assumed that childNodes was a standard collection that could be
indexed by name/id as well as ordinal. However, I just thought that
'calDateText' is an id, so why don't you just use

document.getElementById( 'calDateText' ).innerHTML

?

Mike
 
L

Lasse Reichstein Nielsen

I have a function that works with explorer but not with netscape.
The problem is the function at line 5.

You don't say which Netscape. If it's Netscape 4, your options are
very limited. If it's Netscape 6+, i.e., based on Mozilla, then
you can pretty much fly :) I'll assume Netscape 6+.
5 var elementText = eval(myElement.children["calDateText"].innerText);

I see three problems:
- innerText is IE only.
- children is IE only.
- eval is evil.

There are different options, depending on how standards compliant you
want to be. I'll go for full compliance with the DOM specification
(i.e., avoiding innerHTML).

A version that collects the string content of the calcDateText element
(not recursively, so don't put the text to find inside tags).
---
var span = document.getElementById("calcDateText");
var elementText = "";
for(var chld = span.firstChild; chld; chdl=chld.nextSibling) {
if (chld.nodeType == 3) { // text node
elementText += chld.nodeValue;
}
}
---
Or, change the first line to:
---
var span = myElement.getElementsByTagName("span")[0];
---
(Notice that I use "span" instead of "font" as commented later)
11 <td id=calCell onclick='fSetSelectedDay(this)'>
12 <font id='calDateText' onclick='fSetSelectedDay(this)'>

The font tag is deprecated and you are not using any of its specific
13 <script> myMonth[w][d] </script>

The type attribute is required on script tags, so:
---
<script type="text/javascript">
---
Do you mean to document.write the value?
---
document.write(myMonth[w][d]));
---
Where are "w" and "d" calculated? (Meaning Week and Day?)
14 </font>
</span>

/L
 
J

Julia Peterwitz

Lasse Reichstein Nielsen said:
You don't say which Netscape. If it's Netscape 4, your options are
very limited. If it's Netscape 6+, i.e., based on Mozilla, then
you can pretty much fly :) I'll assume Netscape 6+.

yes, I thought about netscape 6+
5 var elementText = eval(myElement.children["calDateText"].innerText);

I see three problems:
- innerText is IE only.
- children is IE only.
- eval is evil.

There are different options, depending on how standards compliant you
want to be. I'll go for full compliance with the DOM specification
(i.e., avoiding innerHTML).

A version that collects the string content of the calcDateText element
(not recursively, so don't put the text to find inside tags).

until here it works.
but the following doesn't start.
and I don't know why.
but there is no error message.
for(var chld = font.firstChild; chld; chdl=chld.nextSibling) {
if (chld.nodeType == 3) { // text node
elementText += chld.nodeValue;
}
}
---


The font tag is deprecated and you are not using any of its specific
attributes anyway. Change it to a <span> instead, that is just what
you need: a meaningless wrapper.

I removed the irrelevant attributes.
so I will use said:
13 <script> myMonth[w][d] </script>

The type attribute is required on script tags, so:

sorry I forgot to put the document.write around

13 <script> document.write(myMonth[w][d]); </script>

this function notes the days of the months, but this is irrelevant for
the problem. isn't it?
 
L

Lasse Reichstein Nielsen

until here it works.
but the following doesn't start.
and I don't know why.
but there is no error message.

Damn! :(
That would be the typo: ^^^^ shoud be chld.

That would mean that the loop goes on forever ... doing nothing.

/L
 
J

Julia Peterwitz

/*
....
*/

alert("1");
var font = myElement.getElementsByTagName("font");
alert("2");
var elementText = "";
alert("3");
for(var chld = font.firstChild; chld; chld=chld.nextSibling) {
alert("4");
if (chld.nodeType == 3) { // text node
elementText += chld.nodeValue;
}

/*
....
*/

when I start this I'll get the alert's 1,2,3 but the alert 4 doesn't start.
so I think there is something wrong.
but I don't know what.
 
J

Julia Peterwitz

Michael Winter said:
myElement.childNodes[ 'calDateText' ].innerHTML;
is also not working
(it neither works in explorer nor in netscape)

any other proposals for netscape??

Oops. I assumed that childNodes was a standard collection that could be
indexed by name/id as well as ordinal. However, I just thought that
'calDateText' is an id, so why don't you just use

document.getElementById( 'calDateText' ).innerHTML

?

Mike


elementText = document.getElementById('calDateText').innerHTML;
alert(elementText);

good idea, but elementText is "".
do you know why?
 
Z

ZER0

elementText = document.getElementById('calDateText').innerHTML;
alert(elementText);
good idea, but elementText is "".
do you know why?

Try something like that:

<script type="text/javascript">
function getInnerText(el){
return
el.innerHTML.replace(/(<script[^>]*?>.*?<\/script>)|[\r\n]/gi,"").replace(/<[\/\!]*?[^<>]*?>/g,"");
}

function fSetSelectedDay(el){
alert(">"+getInnerText(el)+"<");
}
</script>

<font id='calDateText'
onclick='fSetSelectedDay(this)'><script>document.write('some text,<br />
maybe with <strong>tags</strong>');</script></font>

I hope it helps.


--
C'ya,
ZER0 :: coder.gfxer.webDesigner();

Il computer e' una macchina progettata per velocizzare e automatizzare
gli errori.
 
M

Michael Winter

[snip]
elementText = document.getElementById('calDateText').innerHTML;
alert(elementText);

good idea, but elementText is "".
do you know why?

Not a clue. Can you post a complete sample (preferably a URL)?

Mike
 
T

Thomas 'PointedEars' Lahn

Julia said:
alert("1");
var font = myElement.getElementsByTagName("font");
alert("2");
var elementText = "";
alert("3");
for(var chld = font.firstChild; chld; chld=chld.nextSibling) { ^
alert("4");
if (chld.nodeType == 3) { // text node ^
elementText += chld.nodeValue;
} ^
/*
...
*/

when I start this I'll get the alert's 1,2,3 but the alert 4 doesn't start.
so I think there is something wrong.
but I don't know what.

Looks like there is a closing brace missing. Does the JavaScript
Console tell you anything? If yes, what does it say?


PointedEars
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

No members online now.

Forum statistics

Threads
473,744
Messages
2,569,484
Members
44,903
Latest member
orderPeak8CBDGummies

Latest Threads

Top