Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ошибка в доке о classList #4188

Open
HellSquirrel opened this issue Jan 5, 2023 · 6 comments
Open

Ошибка в доке о classList #4188

HellSquirrel opened this issue Jan 5, 2023 · 6 comments
Labels
js Контент по JavaScript дока Справочный материал ошибка Что-то критичное и мешает хороший старт Подходит для начинающих

Comments

@HellSquirrel
Copy link
Member

Ссылка на статью

https://doka.guide/js/element-classlist/

Описание

В статье есть следующее утверждение:
У classList есть множество методов, но мы рассмотрим только часто используемые.

Однако методов не множество, вот тут можно посмотреть все что есть
https://dom.spec.whatwg.org/#domtokenlist

Предлагается дополнить изменить статью так, чтобы она перечисляла все методы

Пруфы

No response

@HellSquirrel HellSquirrel added js Контент по JavaScript ошибка Что-то критичное и мешает хороший старт Подходит для начинающих labels Jan 5, 2023
@punkmachine
Copy link
Contributor

В статье не хватает только методов contains() и supports(), согласно спеке, я прав же?

@HellSquirrel
Copy link
Member Author

все так :) contains уже принесли #4184

@punkmachine
Copy link
Contributor

supports() на счёт него в pull request написать, чтобы его добавили? а то в pull request-е только про contains()

@SergeyK2004
Copy link
Contributor

Давайте я и добавлю последний оставшийся метод и исправлю шапку.

@SergeyK2004
Copy link
Contributor

Я почитал про supports() и что-то в сомнениях, надо ли его добавлять.
По сути к classList он не имеет никакого отношения. Используется только для объектов типа HTML DOMTokenList но не для всех, а только некоторых. Я нашел только один пример, это атрибут sandbox у <iframe>.
Применение метода supports() в данном случае такое, он даст истину если ключ в параметре можно использовать как параметр sandbox
Но при использовании supports() на неподдерживаемых элементах, например classList он всегда будет давать TypeError

@SergeyK2004
Copy link
Contributor

Если уж говорить о полноте описания я бы добавил тогда value и может быть toString так как они прописаны в прототипе и являются методами именно classList.
Напишите если нужно.

@TatianaFokina TatianaFokina added this to the Бэклог JS milestone Oct 30, 2023
@TatianaFokina TatianaFokina added the дока Справочный материал label Oct 30, 2023
@TatianaFokina TatianaFokina changed the title Ошибка в статье о classList Ошибка в доке о classList Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
js Контент по JavaScript дока Справочный материал ошибка Что-то критичное и мешает хороший старт Подходит для начинающих
Projects
None yet
Development

No branches or pull requests

4 participants