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

Tiny new features, more robust localization #35

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

FliiFe
Copy link

@FliiFe FliiFe commented May 17, 2022

What changed:

  • UI Localization now goes through Localizable strings. This is to avoid having to propagate changes on Base UI to all languages manually, since the UI is simple enough to not have to make language-specific UI choices
  • Added an option to show battery percentage instead of "Calculating..." when the remaining time is yet-to-be-determined.
  • Split option to show remaining time in two separate options depending on wether the computer is on battery or on AC power
  • Began making the notification system more customisable: all percentages are available
  • Stop depending on Carthage (LaunchAtLogin is deprecating its Carthage support, and Xcode handles dependencies well)

Please let me know if there are things that should be reviewed further

What changed:
- UI Localization now goes through Localizable strings. This is to avoid having to propagate changes on Base UI to all languages manually, since the UI is simple enough to not have to make language-specific UI choices
- Added an option to show battery percentage instead of "Calculating..." when the remaining time is yet-to-be-determined.
- Split option to show remaining time in two separate options depending on wether the computer is on battery or on AC power
- Began making the notification system more customisable: all percentages are available (missing: modal to add a percentage, working: deleting percentages from the list)
- Stop depending on Carthage (LaunchAtLogin is deprecating its Carthage support, and Xcode handles dependencies well)
@FliiFe FliiFe marked this pull request as ready for review May 17, 2022 14:44
@FliiFe
Copy link
Author

FliiFe commented May 19, 2022

This PR resolves #21, resolves #27 and resolves #28

@decodism decodism mentioned this pull request Jun 28, 2022
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

1 participant