Solved

<a> tag's "onclick" event not working

Posted on 2009-07-07
18
354 Views
Last Modified: 2012-05-07
in the attached code snippet, onclick event is not working.

When i click on the "SUBMIT" link, i'm able to see the alert box with message "Came HERE". But, "frmNew" is not submitted.
Please let me know if i need to do anything in "fun()" without changing anything in the anchor tag <a>
<a id="errBoxCloseLink" href="javascript:void(0);" onclick="fun()" style="color:white;font-weight:bolder;text-decoration:none" title="Close Errors" linkcontrol="ignore">SUBMIT</a>

 

<script>

function fun()

{

    alert("Came HERE");

    document.forms["frmNew"].submit();

}

</script>

Open in new window

0
Comment
Question by:Ravi Kalla
  • 3
  • 3
  • 2
  • +3
18 Comments
 
LVL 7

Expert Comment

by:Valleriani
ID: 24793209
Are you defining the form name?

<form name="frmNew">
<a id="errBoxCloseLink" href="javascript:void(0);" onclick="fun()" style="color:black;font-weight:bolder;text-decoration:none" title="Close Errors" linkcontrol="ignore">SUBMIT</a>
</form>

<script>
function fun()
{
    alert("Came HERE");
    document.forms["frmNew"].submit();
}
</script>

?
0
 
LVL 14

Expert Comment

by:shobinsun
ID: 24793259
Hi,

remove the style attribute from the line and use:

<a id="errBoxCloseLink" href="javascript:void(0);" onclick="fun()" class="vClass" title="Close Errors" linkcontrol="ignore">SUBMIT</a>

and then

<style type="text/css">
a.vClass:link {color: blue;}
</style>

Hope this will help you.

Regards
0
 
LVL 14

Expert Comment

by:shobinsun
ID: 24793292
hi,

sorry..you should check if there you defined the form name as commented by  ' Valleriani: '

Regards
0
 
LVL 6

Author Comment

by:Ravi Kalla
ID: 24793381
yes... i have already defined the form name as in the attached code
 

<script>

function fun()

{

    alert("Came HERE");

    document.forms["frmNew"].submit();

}

</script>
 

<a id="errBoxCloseLink" href="javascript:void(0);" onclick="fun()" style="color:black;font-weight:bolder;text-decoration:none" title="Close Errors" linkcontrol="ignore">SUBMIT</a>
 

<form name="frmNew">

</form>

Open in new window

0
 
LVL 7

Expert Comment

by:Valleriani
ID: 24793411
Tested in IE/FF, they seem to be both submitting though. (Page is reloading basicly)

Can we get some more details? Is it tied in another script? Browser version? etc?
0
 

Expert Comment

by:gshahlot
ID: 24793442
You might wanna try running these on some other browsers.
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 
LVL 6

Author Comment

by:Ravi Kalla
ID: 24793520
i'm able to submit by changing the value of "href" to "javascript:fun()"
0
 
LVL 7

Expert Comment

by:Valleriani
ID: 24793550
Hrm, could void(0) be interfering with some browsers? Could try:

<a id="errBoxCloseLink" href="#" onclick="fun()" style="color:white;font-weight:bolder;text-decoration:none" title="Close Errors" linkcontrol="ignore">SUBMIT</a>
0
 
LVL 13

Accepted Solution

by:
Murali Murugesan earned 500 total points
ID: 24793997
in "fun()" add return false; as the last statement.

-Murali*
0
 
LVL 54

Expert Comment

by:b0lsc0tt
ID: 24829750
Ravi_Kalla,
I am going to post this as an objection.  The expert's post at http:#a24793997 has the key to your solution.  You need return false to follow the script or the browser just goes to the href as a natural next step.
The reason I post this as an objection is it isn't correct to have javascript in the href tag.  Depending on the doctype and browser you could even have problems.  Even the "javascript: void(0);" code isn't recommended in href anymore.  It is better to use # (as suggested in http:#a24793550) or even best to provide an html page that will show some "Javascript needs to be supported" message.  The key is the proper fix isn't to move the Javascript but to use return false so the event handling stops.
Let me know if you have a question about this.  It is worth changing your code because, even if the comment above was posted after you started closing this, it is the correct answer and way to do this.
b0lsc0tt
0
 
LVL 13

Expert Comment

by:Murali Murugesan
ID: 24837024
Should there be a points split?

-Murali*
0
 
LVL 6

Author Comment

by:Ravi Kalla
ID: 24869383
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This article demonstrates how to create a simple responsive confirmation dialog with Ok and Cancel buttons using HTML, CSS, jQuery and Promises
Finding original email is quite difficult due to their duplicates. From this article, you will come to know why multiple duplicates of same emails appear and how to delete duplicate emails from Outlook securely and instantly while vital emails remai…
In this tutorial viewers will learn how add a full-size background image to a webpage using CSS3. Create a new HTML document with an internal stylesheet.: In CSS, define the html element to have a background image. Use a high resolution image.: In t…
The viewer will learn the benefit of using external CSS files and the relationship between class and ID selectors. Create your external css file by saving it as style.css then set up your style tags: (CODE) Reference the nav tag and set your prop…

932 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

13 Experts available now in Live!

Get 1:1 Help Now