Skip to content

Change "bubbles" to "bubble" - #675

Merged
LadyBluenotes merged 1 commit into
solidjs:mainfrom
klequis:patch-7
Apr 13, 2024
Merged

LadyBluenotes merged 1 commit into
solidjs:mainfrom
klequis:patch-7

Conversation

@klequis

@klequis klequis commented Apr 11, 2024

Copy link
Copy Markdown
Contributor

Change "bubbles" to "bubble"

  • from: dispatch events to the relevant elements as they bubbles up.
  • to: dispatch events to the relevant elements as they bubble up.

dispatch events

"In this method event listeners are attached to the document element, and dispatch events to the relevant elements as they bubbles up."

This is unclear to me. Is "dispatch events" a thing/noun or is the wording wrong?

Perhaps it should be:

  • ... event listeners are attached to the document element, and dispatched to the relevant elements ...

**Change "bubbles" to "bubble"**

- from: dispatch events to the relevant elements as they bubbles up.
- to: dispatch events to the relevant elements as they bubble up.

**dispatch events**

"In this method event listeners are attached to the `document` element, and dispatch events to the relevant elements as they bubbles up."

This is unclear to me. Is "dispatch events" is a thing/noun which I have not heard of?

Otherwise, it could be saying:
- ... event listeners are attached to the `document` element, and dispatch**ed** to the relevant elements ...
@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify

netlify Bot commented Apr 11, 2024 •

Copy link
Copy Markdown

✅ Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit aac13a9
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/66186ade23bd48000814c696
😎 Deploy Preview https://deploy-preview-675--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LadyBluenotes
LadyBluenotes merged commit b0a5b06 into solidjs:main Apr 13, 2024
ryansolid added a commit that referenced this pull request Oct 15, 2026
…1653)

* reference: regenerate against solid next eb4e0192; dispositions for rc.14-rc.15 exports

New pages: markRaw, onHydrationEnd, dispatchAsInteraction. dynamicComponent,
lazyModule and reportRequestFailure join the dynamic, lazy and
configureServerErrors pages; ssrStyleProperties is compiler ABI and hidden.
The NoArray brand on the seedless derived-store form is dropped from
rendered signatures.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y4HXPRqXPnqriNxDyin6eK

* docs: catch up hand-written pages with solid rc.14-next and router next.34-next.40

Solid: seedless derived stores and complete seeds (#4073); storePath
recipes become draft setters (#4053); respond() envelope and non-JSON
bodies; performance-tracks defaults; kind "request" and
reportRequestFailure; lowercase on* migration note; dynamicComponent,
lazyModule and onHydrationEnd mentions.

Router: link preloading is opt-in (preloadLinks strategies); data-pending
needs pendingLinks; aria-current matches path and query; decoded params
(#681); useInRouter (#675); action settle timing; liveQuery reference;
anchor attributes typed by the router.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y4HXPRqXPnqriNxDyin6eK

* docs(router): say held update, not transition, per the Solid Terms table

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y4HXPRqXPnqriNxDyin6eK

---------

Co-authored-by: Claude <noreply@anthropic.com>
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.

2 participants