Skip to content

pat-inject code optimizations #1222

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Jan 12, 2025
Merged

pat-inject code optimizations #1222

merged 9 commits into from
Jan 12, 2025

Conversation

thet
Copy link
Member

@thet thet commented Jan 12, 2025 •
edited
Loading

@thet thet mentioned this pull request Jan 12, 2025
Closed
@thet thet force-pushed the pat-inject-modernization branch from f995680 to 74d7196 Compare January 12, 2025 22:14
@thet thet mentioned this pull request Jan 12, 2025
Closed
thet added 9 commits January 12, 2025 23:50
@thet
fix(core dom): Do not break querySelectorAllAndMe, if passed element …
1b19844 
…is not a real element but something like a text node.
@thet
pat-inject: Improve code for updating the history object.
3369d3a
@thet
pat-inject: Minor code optimization.
5c3c1aa
@thet
pat-inject: Minor code optimization: Consistently use nodeName over t…
fa07469 
…agName.

nodeName returns for Elements the same value but also works on document
fragments and other type of nodes.
@thet
pat-inject: Minor code optimization: Rework most forEach to for loops.
d5f69c2
@thet
pat-inject: Code modernization: Remove dependency on jquery-ext.
9a55aa3
@thet
pat-inject: Minor code optimization.
cdc346a
@thet
maint(pat inject): Modernize some parts of pat inject.
b0f94fb
@thet
maint(pat-inject): Use create_uuid for generating the temporary autol…
8a3af53 
…oad uuid and remove it again afterwards.
@thet thet force-pushed the pat-inject-modernization branch from 951079c to 8a3af53 Compare January 12, 2025 22:51
@thet thet merged commit 13bf6dc into master Jan 12, 2025
1 check passed
@thet thet deleted the pat-inject-modernization branch January 12, 2025 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@thet