Always reset scroll after durable column is maximized
ClosedPublic
Actions

Authored by chad on Oct 3 2016, 2:42 AM.

Tags

None

Referenced Files

	F16643798: D16649.id40082.diff
	Tue, Jun 17, 10:52 AM

	Unknown Object (File)
	Thu, May 29, 8:38 AM

	Unknown Object (File)
	Apr 25 2025, 2:40 PM

	Unknown Object (File)
	Mar 28 2025, 7:48 AM

	Unknown Object (File)
	Mar 25 2025, 2:54 AM

	Unknown Object (File)
	Mar 25 2025, 2:54 AM

	Unknown Object (File)
	Mar 19 2025, 6:22 AM

	Unknown Object (File)
	Mar 15 2025, 3:09 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rP55a56c09e7fd: Always reset scroll after durable column is maximized

Summary

When the durable column is re-opened, scroll user to latest message. Though later we should scroll them to the last read.

Test Plan

Minimize window, reload page, pop up window, see proper scroll position.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40082.Oct 3 2016, 2:42 AM

chad retitled this revision from to Always reset scroll after durable column is maximized.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptOct 3 2016, 2:42 AM

epriestley accepted this revision.Oct 3 2016, 3:29 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Oct 3 2016, 3:29 AM

Closed by commit rP55a56c09e7fd: Always reset scroll after durable column is maximized (authored by chad, committed by chad). · Explain WhyOct 3 2016, 3:32 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

26 lines

webroot/

rsrc/

js/

application/

conpherence/

behavior-durable-column.js

5 lines

Diff 40085

View Options

resources/celerity/map.php