korgac: hide "Dismiss all" button if there's only one reminder.
ClosedPublic
Actions

Authored by dfaure on Oct 29 2018, 10:01 AM.

Details

Reviewers

winterz
mlaurent

Commits

Summary

It's confusing to have two buttons that do the exact same thing.

Test Plan

Reminder dialog now only shows "Dismiss reminder" when there is only one reminder in the list

Repository

R210 KOrganizer

Branch

Applications/18.08

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 4306
Build 4324: arc lint + arc unit

Restricted Application edited subscribers, added: kde-pim; removed: KDE PIM. · View Herald Transcript

dfaure requested review of this revision.Oct 29 2018, 10:01 AM

This revision is now accepted and ready to land.Oct 29 2018, 10:07 AM

Laurent: can you do the merge to master? There's a whole lot of version numbers conflicts which I don't have time to sort out right now ;)

Yep I will do it soon.
thanks

			Path	Packages
M			korgac/alarmdialog.cpp (2 lines)

Commit	Tree	Parents	Author	Summary	Date
4134080a7102	97b6de407c86	848b916b6ab3	David Faure	korgac: hide "Dismiss all" button if there's only one reminder. (Show More…)	Oct 28 2018, 9:16 PM

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	44413	848b916		Oct 29 2018, 10:01 AM	★	★