New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Article language tweaks for Article "Read More" #258
Conversation
Per Josh's comments on the ticket.
public let titleText: String = localizedStringForKeyFallingBackOnEnglish("article-read-more-title") | ||
let site:MWKSite | ||
init(site: MWKSite) { | ||
self.site = site |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is calling super.init()
optional in swift?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. Interestingly in Swift instance vars need to be set before super init... http://stackoverflow.com/a/24150540
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah - anything that is a let
needs to be set before you call init - since that is a constant (can't change later). for instance you can see the delegate ivar in the toc view controller is just a var
. so it doesn't need to be set.
👍 |
👍 restarted the build, which looks to be failing due to timeout issues. |
Article language tweaks for Article "Read More"
Article language tweaks for Article "Read More"
Hebrew version of "food" article:
Before:
After: