Skip to content

Commit 5a4dbea

Browse files
committed
Add custom titles for Recent
1 parent 312da9f commit 5a4dbea

File tree

4 files changed

+20
-12
lines changed

4 files changed

+20
-12
lines changed

WordPress/Classes/Utility/SharedStrings.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,5 +27,7 @@ enum SharedStrings {
2727

2828
enum Reader {
2929
static let unfollow = NSLocalizedString("reader.button.unfollow", value: "Unfollow", comment: "Reader sidebar button title")
30+
static let recent = NSLocalizedString("reader.recent.title", value: "Recent", comment: "Used in multiple contexts, usually as a screen title")
31+
static let discover = NSLocalizedString("reader.discover.title", value: "Discover", comment: "Used in multiple contexts, usually as a screen title")
3032
}
3133
}

WordPress/Classes/ViewRelated/Reader/ReaderDiscoverHeaderView.swift

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ final class ReaderDiscoverHeaderView: UIView, UITextViewDelegate {
2727
addSubview(stackView)
2828
stackView.pinEdges(insets: ReaderStreamTitleView.preferredInsets)
2929

30-
titleView.titleLabel.text = Strings.title
30+
titleView.titleLabel.text = SharedStrings.Reader.discover
3131
titleView.detailsTextView.attributedText = {
3232
guard let details = try? NSMutableAttributedString(markdown: Strings.details) else {
3333
return nil
@@ -204,7 +204,6 @@ enum ReaderDiscoverChannel: Hashable {
204204
}
205205

206206
private enum Strings {
207-
static let title = NSLocalizedString("reader.discover.header.title", value: "Discover", comment: "Header view title")
208207
static let details = NSLocalizedString("reader.discover.header.title", value: "Explore popular blogs that inspire, educate, and entertain based on your [interests](/interests).", comment: "Reader Discover header view details label. The text has a Markdown URL: [interests](/interests). Only the text in the square brackets needs to be translated: [<translate_this>](/interests).")
209208
static let editInterests = NSLocalizedString("reader.editInterests.title", value: "Edit Interests", comment: "Screen title")
210209
}

WordPress/Classes/ViewRelated/Reader/ReaderStreamTitleView.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,18 +22,16 @@ final class ReaderStreamTitleView: UIView {
2222
fatalError("init(coder:) has not been implemented")
2323
}
2424

25-
static let preferredInsets = UIEdgeInsets(top: 16, left: 16, bottom: 8, right: 16)
25+
static let preferredInsets = UIEdgeInsets(top: 4, left: 16, bottom: 8, right: 16)
2626
}
2727

2828
extension ReaderStreamTitleView {
2929
static func makeForFollowing() -> ReaderStreamTitleView {
3030
let view = ReaderStreamTitleView()
31-
view.titleLabel.text = Strings.followingTitle
31+
view.titleLabel.text = SharedStrings.Reader.recent
3232
view.detailsTextView.text = Strings.followingDetails
3333
return view
3434
}
35-
36-
static var followingTitle: String { Strings.followingTitle }
3735
}
3836

3937
final class ReaderNavigationCustomTitleView: UIView {
@@ -68,6 +66,5 @@ final class ReaderNavigationCustomTitleView: UIView {
6866
}
6967

7068
private enum Strings {
71-
static let followingTitle = NSLocalizedString("reader.following.header.title", value: "Following", comment: "Screen header title")
7269
static let followingDetails = NSLocalizedString("reader.following.header.details", value: "Stay current with the blogs you've subscribed to.", comment: "Screen header details")
7370
}

WordPress/Classes/ViewRelated/Reader/ReaderStreamViewController.swift

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -641,11 +641,21 @@ import AutomatticTracks
641641
}
642642

643643
if FeatureFlag.readerReset.enabled {
644-
if ReaderHelpers.topicIsFollowing(topic) {
645-
title = ReaderStreamTitleView.followingTitle
646-
titleView.textLabel.text = ReaderStreamTitleView.followingTitle
647-
navigationItem.titleView = titleView
648-
}
644+
configureCustomTitleView(for: topic)
645+
}
646+
}
647+
648+
private func configureCustomTitleView(for topic: ReaderAbstractTopic) {
649+
var title: String?
650+
if ReaderHelpers.topicIsFollowing(topic) {
651+
title = SharedStrings.Reader.recent
652+
} else if ReaderHelpers.topicIsDiscover(topic) {
653+
title = SharedStrings.Reader.discover
654+
}
655+
if let title {
656+
self.title = title
657+
titleView.textLabel.text = title
658+
navigationItem.titleView = titleView
649659
}
650660
}
651661

0 commit comments

Comments
 (0)