LearnNewsExamplesServices
Frontmatter
id5900
titlemain.DomEvents: onClick() => limit the message to app to the event.detail property => < 2
stateClosed
labels
enhancement
assigneestobiu
createdAtSep 13, 2024, 6:39 PM
updatedAtSep 15, 2024, 4:23 PM
githubUrlhttps://github.com/neomjs/neo/issues/5900
authortobiu
commentsCount2
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtSep 15, 2024, 4:23 PM

main.DomEvents: onClick() => limit the message to app to the event.detail property => < 2

Closed v8.1.0 enhancement
tobiu
tobiu commented on Sep 13, 2024, 6:39 PM

(No description provided)

tobiu added the enhancement label on Sep 13, 2024, 6:39 PM
tobiu assigned to @tobiu on Sep 13, 2024, 6:39 PM
tobiu referenced in commit 7de976e - "main.DomEvents: onClick() => limit the message to app to the event.detail property => < 2 #5900" on Sep 13, 2024, 6:39 PM
tobiu closed this issue on Sep 13, 2024, 6:39 PM
tobiu
tobiu Sep 13, 2024, 6:51 PM

this is obviously assuming that we want onClick() to act like onSingleClick().

tobiu
tobiu Sep 15, 2024, 4:22 PM

thinking more about this one: in case a new dev wants to create a button click counter increase logic, clicks would get significantly slowed down for no reason, giving a bad performance impression.

so, main should forward all click events, but add the detail (click count) info to the app worker, so that we can work with it there.

tobiu reopened this issue on Sep 15, 2024, 4:22 PM
tobiu referenced in commit 889acab - "#5900 main.DomEvents: onClick() => passing the detail property to app" on Sep 15, 2024, 4:23 PM
tobiu closed this issue on Sep 15, 2024, 4:23 PM