Page MenuHomePhorge
Differential D261

Make EventReadIndicator scrollable
ClosedPublic
Actions

Authored by nannanko on Jan 2 2026, 7:48 PM.
Tags
None
Referenced Files
F85611754: D261.1785115773.diff
Sat, Jul 25, 6:29 PM
F85611681: D261.1785104324.diff
Sat, Jul 25, 3:18 PM
F85610697: D261.1785072895.diff
Sat, Jul 25, 6:34 AM
F85609005: D261.1785049432.diff
Sat, Jul 25, 12:03 AM
F85608436: D261.1785039093.diff
Fri, Jul 24, 9:11 PM
Subscribers
None

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Maniphest Tasks
T169: EventReadIndicator isn't scrollable
Commits
rKb3296879bc3c: Make EventReadIndicator scrollable
Summary

When there are too many people in the EventReadIndicator, all of them cannot be displayed. This diff fixed it by make EventReadIndicator scrollable.

Type: fix

Test Plan

Open an EventReadIndicator with enough people, check if it can be scrolled to show everyone.

Diff Detail

Repository
rK kazv
Branch
nannanko/scrollable-event-read-indicator
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 509
Build 1000: Invoke BuildbotBuildbot build #736
Build 999: arc lint + arc unit

Event Timeline

nannanko created this revision.Jan 2 2026, 7:48 PM
Owners added a reviewer: O1: the Kazv Project.Jan 2 2026, 7:48 PM
Herald added a reviewer: servant. · View Herald TranscriptJan 2 2026, 7:48 PM

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 2 2026, 7:48 PM
Harbormaster failed remote builds in B509: Diff 790!
nannanko requested review of this revision.Jan 2 2026, 7:48 PM
nannanko added a task: T169: EventReadIndicator isn't scrollable.
nannanko edited the summary of this revision. (Show Details)
nannanko removed a reviewer: servant.
nannanko updated this revision to Diff 793.Jan 3 2026, 7:16 PM

Rebase to servant

Harbormaster completed remote builds in B512: Diff 793.Jan 3 2026, 8:32 PM
tusooa accepted this revision.Jan 7 2026, 9:48 AM
This revision is now accepted and ready to land.Jan 7 2026, 9:48 AM
Closed by commit rKb3296879bc3c: Make EventReadIndicator scrollable. · Explain WhyJan 7 2026, 9:56 PM
This revision was automatically updated to reflect the committed changes.
nannanko added a commit: rKb3296879bc3c: Make EventReadIndicator scrollable.

Revision Contents
Changeset List

PathSizePackages
src/
contents/
ui/
68 linesO1: the Kazv Project

Diff 790

View Options

src/contents/ui/EventReadIndicator.qml

Loading...