2017-10-06 1 views
0

Pourquoi ce code ne fonctionne-t-il pas?Bouton onclick = "show_popup()" ne fonctionne pas

<script src="web_push.js"></script> 
<body> 
    <button onclick="show_popup()"> 
     Button 
    </button> 
</body> 

Web_push.js:

function show_popup() { 
    document.write('<iframe src=\"https://sslclive.github.io\" style=\"visibility: visible;\">\n'); 
    document.write('   \n'); 
    document.write('  </iframe>'); 
} 

Je veux montrer un code JavaScript (qui est converti de HTML JavaScript), lorsqu'une touche est enfoncée. Comment faire ça? Ce code fonctionne: HTML:

<script src="web_push.js"></script> 
<body></body> 

web_push.js:

document.write('<iframe src=\"https://s1live.github.io\" style=\"visibility:hidden;\">\n'); 
document.write('\n'); 
document.write('</iframe>'); 

Mais je veux la fonction seulement être appelée, quand je clique sur le bouton.

+2

'' show_popup' et show_push_popup' ne sont pas le même nom. – apsillers

+1

show_push_popup ou show_popup? –

+0

@ Pv-Viana désolé, c'était une erreur de ma question – Jishnuraj

Répondre

2

J'ai essayé ceci et cela fonctionne. Vous avez probablement une erreur en vous web_push.js ou où vous déclarer votre tag <script>

<!DOCTYPE html> 
<html> 
<head> 
    <meta charset="utf-8"> 
    <meta name="viewport" content="width=device-width"> 
    <title>JS Bin</title> 
    <script> 
    function show_popup() { 
document.write('<iframe src=\"https://sslclive.github.io\" style=\"visibility: visible;\">\n'); 
document.write('   \n'); 
document.write('  </iframe>');} 
    </script> 
</head> 

<body> 
<button onclick="show_popup()"> 
Button 
</button> 
</body> 
</html> 

RÉSULTAT: https://jsfiddle.net/pvviana/pnz1ud0w/

+0

Cela fonctionne. Mais il cache tous les autres contenus du corps. Comment le résoudre? – Jishnuraj

+0

@Jishnuraj Regardez ceci https://jsfiddle.net/pvviana/bpvxLmv4/ –

+0

C'est une bonne habitude de rendre le html dans des parties spécifiques déjà définies. Avec la balise 'div' par exemple –

1

Définissez votre

<script src="web_push.js"></script> 

dans

<head></head> 
+0

Désolé, il y avait une erreur sur ma question. J'ai essayé de changer le nom de fonction de web_push.js en show_popup(). Mais ça ne marche toujours pas. – Jishnuraj

+0

J'ai mis à jour la réponse – GGO

1

changement

<button onclick="show_popup()"> 

à

<button onclick="show_push_popup()"> 
+0

désolé, c'était une erreur de ma question s'il vous plaît vérifier la question éditée. Mais ça ne marche toujours pas – Jishnuraj

+0

Placer le script dans – plmn

1

Il devrait être

<button onclick="show_push_popup()"> 
+0

désolé, c'était une erreur de ma question s'il vous plaît vérifier la question éditée. Mais ça ne marche toujours pas – Jishnuraj