Opt out of initial default focus. about focus-trap HOT 4 CLOSED

Hi, @hally9k. Thanks for the compliment! This conversation about initialFocus has already happened multiple times. Quick summary: I still don't think it's a good idea. Please search the closed issues and look through those comments, and if you feel like you have new reasons why this might be necessary, please re-open the issue.

from focus-trap.

Hi @davidtheclark,

Ok, so Let me try and outline a scenario in which this is a valid use case.

We are working on an app in which information about a person is collected. A person's info is displayed in a modal which requires a focus trap to function nicely. This modal is used to view the person's info as well as being live for editing that person's data. The viewing use case is more frequent than the editing use case. We are aware that focussing the initial input when the user is merely viewing the info opens the door to potential unintentional edits and lose precious info.

Below is a screen shot to illustrate the scenario where a user might only be viewing info but in danger of unintentionally editing it. As you might glean from the image there are certain scenarios in which this loss of data could have a high impact.

from focus-trap.

@hally9k: I think in this case you should put initial focus on the modal container itself (with a tabindex of -1). Demo 4 exemplifies this: https://davidtheclark.github.io/focus-trap/demo/

from focus-trap.

@davidtheclark Thanks for the prompt reply. How would I achieve your suggested solution using your react wrapper? Apologies if I am overlooking something obvious...

from focus-trap.