Is it a live page that we can look at ? Would hate to give up now....
Either that or upload the code as a text file....

    My version works perfect in my case-system... :p

    I think its a real "copy&paste"-Problem :rolleyes: ive changed my post 10² times... always java_script instead of "oneword" javescript...

    (del "e" set "a" - just a test 🆒 )

      Regarding my code above:
      It does work, but as the venerable Weedpacket points out, the bloody forum has chewed up "javascript" into two words "java" and "script". Fix that an' it should work - I took the code from a site I built and tested on about 8 browsers.

        Make sure you dont have any programs that block pop ups !

        :p

          Hi,

          Thanks a lot for encouraging me not to give up.

          First to the code of daveyboy:

          I understood what our venerable master of the telecaster mr. Weedpacket was saying.
          I had changed it to "javascript" but ended up with a "js error". As I'm not allowed to put netscape on my pc, with IE I'm unable to view the js error.

          Secondly :
          Work with pop-up killer and have it properly configured, my other pop-ups work fine.

          Thirdly:
          Is not on a server, but I attach my news.inc (in txt) here.

          Have to go to work now ...

          And thanks again for helping me !

          Gijs

            Originally posted by gijs
            I understood what our venerable master of the telecaster mr. Weedpacket was saying.
            I had changed it to "javascript" but ended up with a "js error". As I'm not allowed to put netscape on my pc, with IE I'm unable to view the js error.

            You may quote this post in a case to allow it 🙂

            I took your attachment, stripped out the PHP (replacing the bits that echo stuff with boilerplate) to make a plain HTML file - the sort that would be arriving at the browser. I ran it in Mozilla, and the Javascript console announced:

            Error: Syntax Error
            Source file: news.inc.html Line 14
            }

            Sure enough the function definition

            function OpenNews( newsid )
{
	var newswin = window.open( 'news/news_display.php?news_id=' + newsid, 
			'newswin', 'toolbar=no, location=no, directories=no, \
			status=no, menubar=no, scrollbars=yes, resizable=yes, \
			copyhistory=no, width=600, height=450')			
		window.blur()
		newswin.focus()
	}
}

            has a spurious } on the end. Took that out and it worked.

              <nuff said>

              If only our eyes were as good at seeing them as a computer...

                Indeed it does. I beg forgiveness - a relic of the if...else clause I removed. The original code checked to see if it was already in "newswin" in which case it didn't run window.open().

                Here's the original:

                function NoteAction( action, nid )
{
	var url = '<!--WEBROOT-->notes.php?action=' + action + '&nid=' + nid
	NoteWin( url )
}

function NoteWin( url )
{
	if ( window.name == 'noteswin' ) {
		window.location.href = url
	} else {
		var noteswin = window.open( url, 
			'noteswin', 'toolbar=no, location=no, directories=no, \
			status=no, menubar=no, scrollbars=yes, resizable=yes, \
			copyhistory=no,width=600,height=450')			
		window.blur()
		noteswin.focus()
	}
}

                I shall go and stand in the corner now.

                  Your forgiveness has been granted.. you may come out of the corner and help others forthwith.

                  🆒

                    Originally posted by The Chancer
                    <nuff said>

                    If only our eyes were as good at seeing them as a computer...

                    Indeed (I didn't even bother looking, to be honest).

                    Now, if only the computer was as good at correcting them as we are... 😃

                      Supreme councel,

                      I humbly thank you all in an alfanumeric way: daveyboy, jdh
                      KoshNaranek, The Chancer, tomhath and of course the venerable Master of the Telecaster Mr. Weedpacket.

                      BTW: as I'm not a native English speaking person, I'm not quite sure what to make of:

                      You may quote this post in a case to allow it 🙂

                      But if I'm correct, my answer would be: it's GNU-licensed 😃

                      Again, thanks a lot for the help !

                      Cheers,

                      Gijs

                        Write a Reply...