Differential D16650

Don't mark a thread as seen if durable column is minimized
ClosedPublic
Actions

Authored by chad on Oct 3 2016, 3:13 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Apr 22, 10:42 PM

	Unknown Object (File)
	Tue, Apr 16, 5:09 PM

	Unknown Object (File)
	Thu, Apr 11, 7:28 AM

	Unknown Object (File)
	Wed, Apr 10, 8:18 PM

	Unknown Object (File)
	Mon, Apr 1, 8:19 AM

	Unknown Object (File)
	Mar 18 2024, 4:40 AM

	Unknown Object (File)
	Feb 17 2024, 2:09 PM

	Unknown Object (File)
	Feb 14 2024, 10:52 PM

View All 100 Files

Subscribers

Details

Reviewers

Maniphest Tasks

T11726: Durable Column "minimized" should notify you of new messages in rooms

Commits

rPda1ed2c63ac9: Don't mark a thread as seen if durable column is minimized

Summary

More work to do here on the JS side, but this at least makes sure users with a small chat window have some notification marked that new replies have not been seen.

Test Plan

Open two windows. Window 1 has durable minimized, Window 2 is full conpherence. Send a message from Window 2, see header count in Window 1 increase. Repeat with durable open, see no change in window.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40083.Oct 3 2016, 3:13 AM

chad retitled this revision from to Don't mark a thread as seen if durable column is minimized.

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, 3:13 AM

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

epriestley edited edge metadata.

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

Closed by commit rPda1ed2c63ac9: Don't mark a thread as seen if durable column is minimized (authored by chad, committed by chad). · Explain WhyOct 3 2016, 3:31 AM

This revision was automatically updated to reflect the committed changes.

chad added a task: T11726: Durable Column "minimized" should notify you of new messages in rooms.Oct 3 2016, 6:04 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

controller/

ConpherenceUpdateController.php

9 lines

Diff 40084

src/applications/conpherence/controller/ConpherenceUpdateController.php

Loading...