Skip to content

Add withPreventDoubleClick HoC#2122

Closed
rickycodes wants to merge 7 commits intodevelopfrom
add-prevent-doubleclick-hoc
Closed

Add withPreventDoubleClick HoC#2122
rickycodes wants to merge 7 commits intodevelopfrom
add-prevent-doubleclick-hoc

Conversation

@rickycodes
Copy link
Copy Markdown
Contributor

Description

This makes it so you can't double click in Settings (this component might be useful elsewhere?)

Before:
(you could double click on a setting and it'll push the same thing to the navigation stack twice)

Screen.record.from.2021-01-18.18.43.59.mp4

After:
(accidentally double clicking only pushes one thing to the stack)

Screen.record.from.2021-01-18.18.44.34.mp4

Checklist

  • Tests are included if applicable

@rickycodes rickycodes requested a review from a team as a code owner January 18, 2021 23:48
@rickycodes rickycodes added the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Jan 19, 2021
Copy link
Copy Markdown
Member

@wachunei wachunei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rickycodes rickycodes force-pushed the add-prevent-doubleclick-hoc branch from 1f60f52 to 43dd367 Compare January 19, 2021 16:06
@andrepimenta andrepimenta removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Jan 28, 2021
@rickycodes rickycodes force-pushed the add-prevent-doubleclick-hoc branch from 43dd367 to 2e85699 Compare January 28, 2021 15:11
@rickycodes rickycodes added the needs-qa Any New Features that needs a full manual QA prior to being added to a release. label Jan 28, 2021
@rickycodes
Copy link
Copy Markdown
Contributor Author

closing in favour of: #2191

@rickycodes rickycodes closed this Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-qa Any New Features that needs a full manual QA prior to being added to a release. next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants