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
About page #352
About page #352
Conversation
ui/about_page.go
Outdated
versionValueLabel: common.theme.Body2("v1.5.2"), | ||
buildDateLabel: common.theme.Body1("Build date"), | ||
buildDateValueLabel: common.theme.Body2("2020-09-10"), | ||
networkLabel: common.theme.Body1("Network"), |
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.
try to avoid naming your variable after its type. Your variable name should describe its content not its type. network would just be fine here.
ui/about_page.go
Outdated
@@ -24,16 +69,108 @@ func (win *Window) AboutPage(common pageCommon) layout.Widget { | |||
} | |||
|
|||
// main settings layout |
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.
kindly add a more appropriate comment or if remove it if its unnecessary
ui/about_page.go
Outdated
line: common.theme.Line(), | ||
container: &layout.List{Axis: layout.Vertical}, | ||
versionLabel: common.theme.Body1("Version"), | ||
versionValueLabel: common.theme.Body2("v1.5.2"), |
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.
This should have a text size of 16sp, Body2 has a size of 14sp.
This implements about page of the new design.
Fixes #350