Paths

Table of Contentst

[web] Disabled auto focus on overlay when entering edit mode
AbandonedPublic
Actions

Authored by • kuba on May 16 2023, 5:45 AM.

Tags

None

Referenced Files

	F3352090: D7838.diff
	Sat, Nov 23, 4:35 AM

	Unknown Object (File)
	Wed, Nov 13, 6:11 PM

	Unknown Object (File)
	Tue, Nov 12, 12:37 AM

	Unknown Object (File)
	Wed, Nov 6, 5:15 PM

	Unknown Object (File)
	Mon, Nov 4, 8:18 AM

	Unknown Object (File)
	Mon, Nov 4, 8:18 AM

	Unknown Object (File)
	Wed, Oct 30, 12:21 PM

	Unknown Object (File)
	Oct 2 2024, 6:02 AM

Subscribers

Details

Reviewers

michal
inka

Summary

We want to focus in the textarea instead of the modal. I disabled auto focusing on modal, when the modal is used for editing message.

Test Plan

Checked if after entering edit mode the focus is set to the textarea.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• kuba created this revision.May 16 2023, 5:45 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMay 16 2023, 5:45 AM

• kuba added a parent revision: D7837: [web] Added FocusTrap for modals.May 16 2023, 5:45 AM

Harbormaster completed remote builds in B19392: Diff 26530.May 16 2023, 6:03 AM

• kuba requested review of this revision.May 16 2023, 6:03 AM

• kuba added a child revision: D7849: [web] Prevent user from closing the tab with unsaved edits.May 17 2023, 3:23 AM

Rebase

Harbormaster completed remote builds in B19481: Diff 26646.May 18 2023, 8:39 AM

Why do we want to do that?

In D7838#235423, @inka wrote:

Why do we want to do that?

Ok, I don't think it is necessary now after I added FocusTrap. Canceling the diff.

Revision Contents
Changeset List

Path

Size

lib/

components/

modal-overlay.react.js

7 lines

web/

chat/

edit-message-provider.js

6 lines

Diff 26646

View Options

lib/components/modal-overlay.react.js