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
Update currency and language setting on the Setting page #626
Conversation
- get setting of language and currency when start app
0eee948
to
9898ea5
Compare
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 methods are no longer needed.
godcr/ui/preference/list_preference.go
Line 77 in 9898ea5
func (lp *ListPreference) PostiveButton(strkey string, clicked func()) *ListPreference { |
godcr/ui/preference/list_preference.go
Line 84 in 9898ea5
func (lp *ListPreference) NegativeButton(strkey string, clicked func()) *ListPreference { |
godcr/ui/preference/list_preference.go
Line 187 in 9898ea5
func (lp *ListPreference) layoutButtons() []layout.FlexChild { |
ui/preference/list_preference.go
Outdated
titleStrKey string | ||
items map[string]string //[key]str-key | ||
itemKeys []string | ||
isAutoUpdate bool |
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 is redundant. What exactly is it supposed to be used for?
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.
It use to add title for modal
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.
I was referring to the isAutoUpdate
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 PR failed github build...
Kindly run go fmt ./...
it should resolve the issue.
ui/preference/list_preference.go
Outdated
lp.setValue(lp.optionsRadioGroup.Value) | ||
lp.IsShowing = false | ||
lp.updateButtonClicked() | ||
|
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.
Remove whitespace
60d186f
to
a460b46
Compare
…d#626) * remove cancel and confirm buttons from the setting page language and currency modal - get setting of language and currency when start app - remove methods redundant
This PR resolve #603