Skip to content
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

(related to #86) Updating string keys for AppView.swift #121

Merged
merged 48 commits into from Jul 31, 2022
Merged

(related to #86) Updating string keys for AppView.swift #121

merged 48 commits into from Jul 31, 2022

Conversation

Wind-Explorer
Copy link
Contributor

I am certain that I have updated the file in #86 before but maybe the changes are lost during rebase or resolving conflicts. This PR brings back the lost changes.

Revamped the entire localizations with up-to-date sentences written with the current situation in mind
Lots of old translations no longer apply, and it is a good idea to get people to translate for the app again according to the current situation now that the base localization (English) is accurate and up-to-date
Fixed some issues with the locale files
# Conflicts:
#	PlayCover/ViewModel/Log.swift
#	PlayCover/de.lproj/Localizable.strings
#	PlayCover/en.lproj/Localizable.strings
#	PlayCover/es.lproj/Localizable.strings
#	PlayCover/fr.lproj/Localizable.strings
#	PlayCover/id.lproj/Localizable.strings
#	PlayCover/it.lproj/Localizable.strings
#	PlayCover/ja.lproj/Localizable.strings
#	PlayCover/ko.lproj/Localizable.strings
#	PlayCover/ru.lproj/Localizable.strings
#	PlayCover/vi.lproj/Localizable.strings
#	PlayCover/zh-Hans.lproj/Localizable.strings
# Conflicts:
#	PlayCover/View/AppSettingsView.swift
# Conflicts:
#	PlayCover/View/AppSettingsView.swift
#	PlayCover/ViewModel/Log.swift
# Conflicts:
#	PlayCover/View/AppsView.swift
Reverted button style, brought back "soundAlert.*"
Adjusted minimum window size for fitting to 4 columns of apps just right, fixed a typo that exposed raw string key, and added some horizontal paddings to search bar for better aesthetics

Modded various UI aspect of things

used same style as every other buttons in UI for consistency

Trimmed paddings on bottom bar

Adjusted padding for even paddings around buttons (because there is an un-removable safe area padding at the bottom of the window (which i cant seem to nuke by "ignoreSafeArea()") )

Added a thin border to search bar

for aesthetic purposes and ease of use by creating that contrast between search bar and background
:chef_kiss:

Updated my Chinese locale

updated to the new formatting used
Added spacings to separate different types of messages and changed some keys for consistency

Tuned ".ipa" parts for consistency

Updated text in setup view
moved crash button to the left of "Experiencing issues?" button to save some space and avoid taking up unnecessary row of space just for the 1 button
SwiftUI is all we need ✨
According to Isaac's comments
# Conflicts:
#	PlayCover/View/MainView.swift
… feature/revamped-string-keys

# Conflicts:
#	PlayCover/View/AppSettingsView.swift
#	PlayCover/View/PlayAppView.swift
Added and mapped keys to strings

Updated more UI to work with locale

Reverted signing settings

Reverted changes

Updated locale
mainly the bottom two buttons in swttings view
Updated locale to work with latest PR
mainly the bottom two buttons in swttings view
… feature/revamped-string-keys

# Conflicts:
#	PlayCover/View/AppsView.swift
This reverts commit de00985.

Reverted ignore settings
… feature/revamped-string-keys

# Conflicts:
#	PlayCover/View/ChangeGenshinAccountView.swift
#	PlayCover/View/DeleteGenshinStoredAccountView.swift
#	PlayCover/View/PlayAppView.swift
#	PlayCover/View/StoreGenshinAccountView.swift
# Conflicts:
#	PlayCover/View/AppSettingsView.swift
@Depal1 Depal1 merged commit b15c248 into PlayCover:develop Jul 31, 2022
@Wind-Explorer Wind-Explorer deleted the feature/revamped-string-keys branch August 1, 2022 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants