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

replaced #foo with .foo #274

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ahelmi365
Copy link

Since it is not recommended to use the same ID for multiple HTML elements on the same page, and since the purpose of an ID attribute is to uniquely identify a single element within an HTML document. Using the same ID for multiple elements would violate this principle and can lead to various issues. So considering this rule, the query() function will return one element with the give id #foo, which means that the for loop will iterate once. So I've changes the selector to be .foo (class selector) so we may get more than one element with the same class name "foo".

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.

None yet

1 participant