Почему атрибут onclick не работает?

Question

Почему атрибут onclick не работает?

0

Я пытаюсь сделать функцию на PHP, подобную команде JavaScript alert(), но атрибут onclick OK не работает!

Вот мой код:

<!DOCTYPE html>
<?php
    function alert($title, $text) {
        $html = '<div id="alert" style="background-color:lightGray; text-align:center; height:500px; width:500px; color: black; position:fixed; top: 50%; left:50%; margin-left:-250px; margin-top:-250px;">';
        $html = $html . '<h1 style="background-color:red; border-radius: 15px;">'. $title . '</h1>' . $text;
        $html = $html . '<br><br><button type="button" style="border-radius:25px; height:50px; width:100px; background-color:white; border:none;" onclick="document.getElementById(\"alert\").style.display=none">OK</button>';
        echo $html;
    }
    alert('Testing', 'Testing <em>testing</em><b>123</b>');
?>

Что не так? Я получаю "поле оповещения", но кнопка "ОК" не работает!

Oliver Ni 28 нояб. 2013, в 18:23

Источник

0

если вы повторяете это несколько раз, у вас будет несколько div с одним и тем же идентификатором, и только document.getElementById() вернет только первый найденный идентификатор.
Marc B 28 нояб. 2013, в 17:22
0

Это несоответствие цитаты в селекторе идентификаторов!
adeneo 28 нояб. 2013, в 17:23
0

@MarcB поднимает правильную точку. Если у вас есть два окна с предупреждениями на одной странице, они оба закроют одно из них, что странно. Создавайте уникальный идентификатор каждый раз, когда вы отображаете окно предупреждения.
Halcyon 28 нояб. 2013, в 17:27

Показать ещё 1 комментарий

Теги:

php

javascript

html

2 ответа

1

Изменить на:

document.getElementById(\"alert\").style.display=\"none\"

Я думаю, это также показывает, что ваш нынешний подход довольно ужасен. Необходимость добавления всех этих побегов без выделения кода раздражает.

Посмотрите, можете ли вы найти способ сделать это правильно;) (подсказка: вы можете).

Halcyon 28 нояб. 2013, в 15:32

Ещё вопросы

если вы повторяете это несколько раз, у вас будет несколько div с одним и тем же идентификатором, и только document.getElementById() вернет только первый найденный идентификатор.
Это несоответствие цитаты в селекторе идентификаторов!
@MarcB поднимает правильную точку. Если у вас есть два окна с предупреждениями на одной странице, они оба закроют одно из них, что странно. Создавайте уникальный идентификатор каждый раз, когда вы отображаете окно предупреждения.

Tom Chung · Accepted Answer · 2013-11-28T15-08-00.000Z

element.style.display = 'none' вместо просто none

ps просто хочет быть в стороне от концепции javascript

' даст конфликт с существующим разделителем строк. Вы должны избежать этого хотя бы один раз.
Извините, я был глуп и забыл "процитировать это".
Спасибо! Новый вопрос: stackoverflow.com/questions/20272026/…