Document that hardware back button event for overlays does not fire if overlay has backdropDismiss set to false

Open Beginner friendly
#1,498 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
70/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Stale
Domain
documentation

Research direction

Open the hardware back button documentation at the linked URL and compare its overlay behavior with core/src/utils/overlays.ts around line 66. Document that an overlay does not register for hardware back button handling when backdropDismiss is false, and verify that the page accurately explains this priority behavior.

Written by the indexing model from the issue text.

Description

content

URL
https://ionicframework.com/docs/developing/hardware-back-button

What is missing or inaccurate about the content on this page?
Just spent a few hours debugging why my registration for hardware back button, which shows an alert dialog, keeps firing even when the alert dialog was displayed, essentially showing an alert on top of an alert, although I registered it with priority 1 (which is well bellow the 100 assigned to overlays).

Turns out the overlay is registering for back button handling only if backdropDismiss is set to true, and this is NOT mentioned in the docs at all.
See here for the actual logic.

Dominant language
MDX
Stars
621
Forks
3.2k
Avg merge
1d 2h
Merged PRs (30d)
86

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ionic-team/ionic-docs

All issues in ionic-team/ionic-docs

Similar issues

More Documentation issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.