Skip to content

Commit c41d989

Browse files
committed
Applied INavigableTopicViewModel to IPageTopicViewModel
Since `Title`, `ShortTitle`, and `WebPath` are now defined on both `IPageTopicViewModel` and the new `INavigableTopicViewModel` (adf7ff8), and there's no other "pollution" from any otherwise unnecessary interface requirements on `INavigableTopicViewModel`, we can derive `IPageTopicBindingModel` from `INavigableTopicViewModel`, thus allowing them to share this definition, while also allowing implementations to satisfy both interfaces.
1 parent adf7ff8 commit c41d989

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

OnTopic.ViewModels/_contentTypes/PageTopicViewModel.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ namespace OnTopic.ViewModels {
1818
/// default implementations that can be used directly, used as base classes, or overwritten at the presentation level. They
1919
/// are supplied for convenience to model factory default settings for out-of-the-box content types.
2020
/// </remarks>
21-
public record PageTopicViewModel: TopicViewModel, IPageTopicViewModel {
21+
public record PageTopicViewModel: TopicViewModel, INavigableTopicViewModel {
2222

2323
/*==========================================================================================================================
2424
| SUBTITLE

OnTopic/Models/IPageTopicViewModel.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ namespace OnTopic.Models {
2121
/// provided via the public interface then it will instead need to be defined in some other way.
2222
/// </para>
2323
/// </remarks>
24-
public interface IPageTopicViewModel : ITopicViewModel {
24+
public interface IPageTopicViewModel : ITopicViewModel, INavigableTopicViewModel {
2525

2626
/*==========================================================================================================================
2727
| PROPERTY: META KEYWORDS

0 commit comments

Comments
 (0)