Differential D7299

Expose recurrence to the editor model
ClosedPublic
Actions

Authored by ervin on Aug 13 2017, 11:07 PM.

Details

Reviewers
dfaure
bensi
franckarrecot
Commits
R4:fe3a441392e0: Expose recurrence to the editor model

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin created this revision.Aug 13 2017, 11:07 PM
dfaure accepted this revision.Aug 14 2017, 11:03 AM
dfaure added inline comments.
tests/units/presentation/artifacteditormodeltest.cpp
78

QCOMPARE is better here

This revision is now accepted and ready to land.Aug 14 2017, 11:03 AM
ervin updated this revision to Diff 18142.Aug 14 2017, 3:55 PM

Addressing dfaure's comment

bensi accepted this revision.Aug 15 2017, 8:41 AM
dfaure accepted this revision.Aug 15 2017, 12:26 PM
Closed by commit R4:fe3a441392e0: Expose recurrence to the editor model (authored by ervin). · Explain WhyAug 16 2017, 11:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 18267

View Options

src/presentation/artifacteditormodel.h

Loading...
View Options

src/presentation/artifacteditormodel.cpp

Loading...
View Options

tests/units/presentation/artifacteditormodeltest.cpp

Loading...