Skip to content

bug fix on select span inside input#1

Open
User12987 wants to merge 1 commit intovladilenm:masterfrom
User12987:patch-1
Open

bug fix on select span inside input#1
User12987 wants to merge 1 commit intovladilenm:masterfrom
User12987:patch-1

Conversation

@User12987
Copy link
Copy Markdown

Исправлен баг, который приводил к тому, что событие клика не срабатывало как ожидалось, если клик пришелся не непосредственно по input, а по содержащемуся внутри него span или другому элементу.
Теперь, при клике, проверяется, есть ли в родителях кликнутого элемента нужный input.
Если есть, значит клик пришелся по дочернему элементу нужного input.
Следовательно, пользователь желает открыть выпадающий список.

Исправлен баг, который приводил к тому, что событие клика не срабатывало, если клик пришелся не непосредственно по input, а по содержащемуся внутри него span или другому объекту. Теперь, при клике проверяется, есть ли в родителях кликнутого элемента нужный input.
@User12987
Copy link
Copy Markdown
Author

Кстати, можно сделать лаконичнее:

if (type === 'input' || type === 'value') {
    ...
}

Но, это менее надёжно, потому что помимо value, могут быть и другие элементы.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant