Paths

Table of Contentst

Differential D125

Add confirm popup for delete event
ClosedPublic
Actions

Authored by nannanko on Jul 13 2024, 9:23 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 15, 5:11 AM

	Unknown Object (File)
	Fri, Sep 13, 10:12 AM

	Unknown Object (File)
	Thu, Sep 12, 4:50 AM

	Unknown Object (File)
	Wed, Sep 11, 4:44 AM

	Unknown Object (File)
	Tue, Sep 10, 6:38 PM

	Unknown Object (File)
	Fri, Sep 6, 6:33 AM

	Unknown Object (File)
	Thu, Sep 5, 3:43 PM

	Unknown Object (File)
	Tue, Sep 3, 12:03 AM

Subscribers

None

Details

Reviewers

Group Reviewers

O1: the Kazv Project

Maniphest Tasks

T83: Confirm message deletion

Commits

rK99dfb7d74e57: Add confirm popup for delete event

Summary

This add a confirm overlay for event deletion.

Type: add

Test Plan

Delete an event, verify that the confirm overlay appears.

Diff Detail

Repository

Branch

nannanko/stacked

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 206
Build 401: GitLab CI for kazv
Build 400: arc lint + arc unit

Event Timeline

nannanko created this revision.Jul 13 2024, 9:23 AM

Owners added a reviewer: O1: the Kazv Project.Jul 13 2024, 9:23 AM

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

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

nannanko added a task: T83: Confirm message deletion.Jul 13 2024, 9:23 AM

tusooa edited the summary of this revision. (Show Details)Jul 13 2024, 9:39 AM

tusooa removed a reviewer: servant.

We should also add a prompt for the user to optionally enter a reason for the deletion.

src/contents/ui/Bubble.qml
55–65	use a Component for it. Instantiating OverlaySheets are expensive, so you should only instantiate it on demand. See the other popup for an example.
src/l10n/cmn-Hans/100-ui.ftl
309
src/l10n/en/100-ui.ftl
330
src/tests/quick-tests/tst_EventView.qml
438	the property to test is now `opened`.

Harbormaster completed remote builds in B178: Diff 298.Jul 13 2024, 10:16 AM

nannanko requested review of this revision.Jul 13 2024, 10:16 AM

tusooa requested changes to this revision.Jul 17 2024, 4:14 PM

This revision now requires changes to proceed.Jul 17 2024, 4:14 PM

tusooa added inline comments.Jul 17 2024, 5:09 PM

src/contents/ui/Bubble.qml
55–65	Well, I think we'd better display the content of the event in the popup... See https://iron.lily-is.land/D131 for an example.

Fix translation error
Fix test

nannanko marked 3 inline comments as done.Jul 17 2024, 7:44 PM

Harbormaster completed remote builds in B206: Diff 337.Jul 17 2024, 7:53 PM

Rebase onto D123

Harbormaster completed remote builds in B211: Diff 348.Jul 19 2024, 2:14 AM

Rebase onto servant

Harbormaster completed remote builds in B216: Diff 364.Jul 22 2024, 7:32 PM

Use component for delete event confirm popup
Add event content for delete event confirm popup
Fix test

Harbormaster completed remote builds in B217: Diff 365.Jul 23 2024, 5:43 AM

tusooa requested changes to this revision.Jul 23 2024, 6:04 PM

tusooa added inline comments.

src/contents/ui/RoomTimelineView.qml
108–122 ↗	(On Diff #365)	Don't accept an `eventView` as the property because it can be destroyed at any time, or to refer to another event. Pass the event id directly, and use the EventView inside the overlay to redact it.

This revision now requires changes to proceed.Jul 23 2024, 6:04 PM

nannanko marked 2 inline comments as done.Jul 23 2024, 11:18 PM

Use eventId instead of eventView for signal deleteEventRequested

Harbormaster failed remote builds in B220: Diff 376!Jul 25 2024, 8:26 AM

nannanko planned changes to this revision.Jul 26 2024, 9:21 AM

nannanko requested review of this revision.Jul 26 2024, 9:33 AM

nannanko marked an inline comment as done.

Harbormaster completed remote builds in B220: Diff 376.Jul 26 2024, 9:40 AM

tusooa added inline comments.Jul 26 2024, 12:55 PM

src/contents/ui/RoomTimelineView.qml
107–125 ↗	(On Diff #376)	test this logic

tusooa requested changes to this revision.Jul 26 2024, 5:50 PM

This revision now requires changes to proceed.Jul 26 2024, 5:50 PM

Distinguish between events and local echo, add tests
rebase onto servant

nannanko marked an inline comment as done.Jul 28 2024, 12:55 AM

Harbormaster completed remote builds in B232: Diff 388.Jul 28 2024, 12:58 AM

nannanko planned changes to this revision.Jul 28 2024, 1:34 AM

Use index instead of eventId, fix cant display local echo in confirm popup

tusooa requested changes to this revision.Jul 28 2024, 3:21 PM

tusooa added inline comments.

src/contents/ui/RoomTimelineView.qml
131 ↗	(On Diff #389)	Index is NOT reliable. It can change when new messages come in. Only use event id and txn id, which are invariant.

This revision now requires changes to proceed.Jul 28 2024, 3:21 PM

tusooa added inline comments.Jul 28 2024, 4:28 PM

src/contents/ui/RoomTimelineView.qml
131 ↗	(On Diff #389)	This is exactly the same reason why I told you not to pass an EventView: because the EventView depends on the index.

Use eventId instead of timeline index

Harbormaster failed remote builds in B238: Diff 394!Aug 2 2024, 11:27 PM

nannanko added a parent revision: D156: Add Room::localEcho and related test.Aug 2 2024, 11:56 PM

nannanko marked 2 inline comments as done.

nannanko mentioned this in D143: Enable upload file for reply.Aug 3 2024, 7:25 PM

tusooa requested changes to this revision.Aug 7 2024, 6:56 PM

tusooa added inline comments.

src/tests/quick-tests/tst_RoomTimelineView.qml
200–202 ↗	(On Diff #394)	but deleting local echo can't fail....

This revision now requires changes to proceed.Aug 7 2024, 6:56 PM

nannanko added inline comments.Aug 9 2024, 10:00 PM

src/tests/quick-tests/tst_RoomTimelineView.qml
200–202 ↗	(On Diff #394)	but deleting local echo can't fail.... my mistake.

Remove test code that delete local echo failed

nannanko marked an inline comment as done.Aug 9 2024, 10:24 PM

Harbormaster failed remote builds in B258: Diff 426!Aug 9 2024, 10:34 PM

tusooa accepted this revision.Aug 11 2024, 6:39 PM

This revision is now accepted and ready to land.Aug 11 2024, 6:39 PM

This revision was landed with ongoing or failed builds.Aug 13 2024, 4:07 PM

Closed by commit rK99dfb7d74e57: Add confirm popup for delete event. · Explain Why

This revision was automatically updated to reflect the committed changes.

nannanko added a commit: rK99dfb7d74e57: Add confirm popup for delete event.

Revision Contents
Changeset List

Path

Size

Packages

src/

contents/

ui/

13 lines

O1: the Kazv Project

l10n/

cmn-Hans/

7 lines

O1: the Kazv Project

en/

7 lines

O1: the Kazv Project

tests/

quick-tests/

tst_EventView.qml

3 lines

O1: the Kazv Project

Diff 337

src/contents/ui/Bubble.qml

Loading...

src/l10n/cmn-Hans/100-ui.ftl

Loading...

src/l10n/en/100-ui.ftl

Loading...

src/tests/quick-tests/tst_EventView.qml

Loading...