Skip to content

feat(jsx): #88 inline event handlers support#272

Merged
thescientist13 merged 7 commits into
masterfrom
feat/issue-88-inline-this-references
May 28, 2026
Merged

feat(jsx): #88 inline event handlers support#272
thescientist13 merged 7 commits into
masterfrom
feat/issue-88-inline-this-references

Conversation

@thescientist13
Copy link
Copy Markdown
Member

@thescientist13 thescientist13 commented May 25, 2026 •
edited
Loading

Related Issue

resolves part of #88

Summary of Changes

  1. Support inline event handlers (wraps in an inline anonymous IIFE)
  2. Made sure event object is also passed to this bound event handlers

TODO

  1. handle BlockStatement
  2. remap event object identifier (docs needed?)
  3. host element detection (Light vs Shadow DOM)
  4. add test cases

@thescientist13 thescientist13 self-assigned this May 25, 2026
@thescientist13 thescientist13 added enhancement Improvement to existing functionality JSX documentation Improvements or additions to the website and / or documentation 0.22.0 labels May 25, 2026
@thescientist13 thescientist13 marked this pull request as ready for review May 27, 2026 01:26
@thescientist13 thescientist13 moved this to 👀 In review in [WCC] General Activities May 27, 2026
@thescientist13 thescientist13 linked an issue May 27, 2026 that may be closed by this pull request
Literal this Evaluation and Derivative References (e.g. destructuring) #88
Open
@thescientist13 thescientist13 changed the title feat: #88 inline event handlers feat(jsx): #88 inline event handlers May 28, 2026
@thescientist13 thescientist13 changed the title feat(jsx): #88 inline event handlers feat(jsx): #88 inline event handlers support May 28, 2026
@thescientist13 thescientist13 force-pushed the feat/issue-88-inline-this-references branch from e22800b to 9c18369 Compare May 28, 2026 00:39
@thescientist13 thescientist13 merged commit a7f3b2d into master May 28, 2026
17 checks passed
@thescientist13 thescientist13 deleted the feat/issue-88-inline-this-references branch May 28, 2026 00:41
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in [WCC] General Activities May 28, 2026
@thescientist13 thescientist13 mentioned this pull request May 30, 2026
Draft
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@thescientist13 thescientist13

Labels

0.22.0 documentation Improvements or additions to the website and / or documentation enhancement Improvement to existing functionality JSX

Projects

[WCC] General Activities
Status: ✅ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Literal this Evaluation and Derivative References (e.g. destructuring)

1 participant

@thescientist13