6
votes
<!DOCTYPE html>
<html>
    <head>
    </head>
    <script>    
        function removeP2() {
            var parent = document.getElementById("section");
            var child = document.getElementById("p2");
            parent.removeChild(child);
        }
    </script>
    <body>
        <nav>
            <form>
               <button onclick="removeP2();">Remove</button> 
            </form>
        </nav>
        <section id="section">
            <p id="p1">Paragraph One.</p>
            <p id="p2">Paragraph Two.</p>
        </section>
    </body>
</html>

When I click the submit button, the function executes as usual, but after the function executes, the page reloads itself (removed P2 for a second).

I found a solution which is to delete the "Form" inside the "nav" tag, and it works fine. Just want to ask what is the problem cause that, and if I need to have the "Form" tag inside the "nav" tag, which part I need to fix?

6
try to laod the script under the section and it will work - PacMan
The page reloads because the button submits the form. - nnnnnn
You should also move that script inside your head tags, so that it preloads. It's sitting between the head and body, which is wrong. - ManoDestra

6 Answers

16
votes

The form is getting submitted and your page is reloaded. You need to stop the form submit.

Solution 1: Add a type attribute to your button.

<button type="button" onclick="removeP2();">Remove</button>

This will make sure that the button is not of submit type (which is the default type for buttons inside form tag when type is not specified), and that the form is not submitted on click.

Solution 2: Prevent the Submit event in javascript. So make these changes.

<button onclick="removeP2(event);">Remove</button>

and in the script prevent this event

 function removeP2(event) {
   event.preventDefault(); // prevent the event , ie form submit event
   var parent = document.getElementById("section");
   var child = document.getElementById("p2");
   parent.removeChild(child);
 }

Solution 3: I dont see any need of the form tag over there in the HTML. If its the exact syntax of yours and you really dont have any other elements or purpose with the form submission then you can remove the form tag.

 <nav>
   <button onclick="removeP2();">Remove</button>         
 </nav>
3
votes

Use the code below. It won't submit the form.

<button type="button">My Button</button>
1
votes

See this code:

https://jsfiddle.net/y7mhu3oL/1/

The solution is onsubmit in form tag:

<script>    
    function removeP2() {
        document.getElementById("p2").remove();
        }   
    </script>
<body>
    <nav>
        <form method="POST" onsubmit="removeP2(); return false;">
           <button>Remove</button> 
        </form>
    </nav>
    <section id="section">
        <p id="p1">Paragraph One.</p>
        <p id="p2">Paragraph Two.</p>
    </section>
</body>
1
votes

Because you are not defining a button type, your browser is assuming that the button is type="submit" and attempting to "submit" the form (even though you haven't defined any method or action).

If you add type="button" to the button tag, it overrides the default assumption of submit and won't attempt to submit the form.

jsfiddle: https://jsfiddle.net/hdpk8c3n/

0
votes

event.preventDefault() is the correct answer. When you press a button inside a form, it will automatically submit that form, unless you prevent the default action with event.preventDefault;

function removeP2(event) {
    event.preventDefault()
    var parent = document.getElementById("section");
    var child = document.getElementById("p2");
    parent.removeChild(child);
}
0
votes

Add type="button" to your button tag. In form tag, button's default type is submit. That is why when you click on the button, form is submitted and the page reloads.