Differential D21217

When loading a page with inlines, don't select/focus inlines which we immediately upgrade to "editing"
ClosedPublic
Actions

Authored by epriestley on May 4 2020, 8:37 PM.

Tags

None

Referenced Files

	F15426800: D21217.diff
	Sun, Mar 23, 10:53 AM

	F15407324: D21217.diff
	Tue, Mar 18, 4:54 PM

	F15333503: D21217.diff
	Sat, Mar 8, 2:40 AM

	Unknown Object (File)
	Mon, Mar 3, 2:19 PM

	Unknown Object (File)
	Sun, Mar 2, 7:03 AM

	Unknown Object (File)
	Feb 9 2025, 9:13 AM

	Unknown Object (File)
	Feb 9 2025, 9:13 AM

	Unknown Object (File)
	Feb 9 2025, 9:13 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13513: Make schema changes to inlines to prepare for new inline features
Commits: rP3a76248071eb: When loading a page with inlines, don't select/focus inlines which we…

Summary

Ref T13513. This is a bit clumsy, but the cleanest way to implement "isEditing" inlines today is to send them down as normal inlines and then simulate clicking "edit" on them.

When we do, don't focus the resulting editor: focusing it makes the page scroll around and highlight things in essentially random order as the editors load in.

Test Plan

Reloaded a page with some open editors, wasn't scrolled to them.

Diff Detail

Repository

Branch

focus1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24347
Build 33540: Run Core Tests
Build 33539: arc lint + arc unit

Event Timeline

epriestley created this revision.May 4 2020, 8:37 PM

Harbormaster completed remote builds in B24347: Diff 50530.May 4 2020, 8:38 PM

epriestley requested review of this revision.May 4 2020, 8:38 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 4 2020, 10:15 PM

Closed by commit rP3a76248071eb: When loading a page with inlines, don't select/focus inlines which we… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP3a76248071eb: When loading a page with inlines, don't select/focus inlines which we….

Revision Contents
Changeset List

Path

Size

resources/

celerity/

webroot/

rsrc/

js/

application/

diff/

36 lines

Diff 50530

resources/celerity/map.php

Loading...

webroot/rsrc/js/application/diff/DiffInline.js

Loading...