Skip to content

fix issue #33 - Avoid unused click handlers for images#34

Merged
schlosser merged 6 commits intoschlosser:masterfrom
BePo65:pu/reduceClickHandlers
Oct 10, 2021
Merged

fix issue #33 - Avoid unused click handlers for images#34
schlosser merged 6 commits intoschlosser:masterfrom
BePo65:pu/reduceClickHandlers

Conversation

@BePo65
Copy link
Contributor

@BePo65 BePo65 commented Oct 9, 2021

As a preparation / basis for a pr that solves issue #29 and improves pr #18 this pr avoids unused click handlers for the images (and harmonizes some code styles).

BePo65 added 6 commits October 9, 2021 16:21
consistent (single) quotes
consistent indentation
consistent curly brackets
no dangling commas
Changed default value of onClickHandler setting to null and added click
handler to ProgressiveImage element only, if there is an onClickHandler.
This way no unused click handler is added to image.
@BePo65 BePo65 force-pushed the pu/reduceClickHandlers branch from 34b02f7 to 7d9b67e Compare October 10, 2021 04:59
@schlosser
Copy link
Owner

Thanks for the cleanup!

@schlosser schlosser merged commit 5d4578c into schlosser:master Oct 10, 2021
@BePo65 BePo65 deleted the pu/reduceClickHandlers branch October 11, 2021 04:36
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