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

App improvements #1

Closed
wants to merge 12 commits into from
Closed

App improvements #1

wants to merge 12 commits into from

Conversation

mac-cain13
Copy link
Contributor

I made a number of improvements to your App to make it somewhat more useful for me. The main improvements/thing I added are:

  • Start on login functionality
  • Battery icon
  • Time remaining when charging the battery
  • Textual improvements (less special chars)
  • Some small code improvements

If you want you can merge my improvements into your branch. I think I will improve my version somewhat further in the next days, will send a new pull request by then.

- Removed special chars around the message in the menubar
- Call IOPSGetTimeRemainingEstimate() only once
- Added comments
- Using more properties to improve readability
- Calculcating => Calculating…
- Unlimited => ∞
also updated copyright notice in info plist
We now show the Apple battery icon in the menubar
also ignore XCode userdata/workspace
also added links to contributors and how to contribute
@codler
Copy link
Owner

codler commented Aug 5, 2012

Thank you very much for improving and adding more feature!! I really appreciate it! This is my first mac app so any improvements and suggestions etc are welcome :)

I merged all your commits except one, the "CODE_SIGN_IDENTITY".
I also made some improvements and fixed a bug.

@codler codler closed this Aug 5, 2012
@mac-cain13
Copy link
Contributor Author

Great that you accepted the pull request! I understand why you didn't accept the code sign commit, but you should sign the builds you publish with a Developer ID. If you don't code sign builds Gatekeeper will prevent users from installing it!

@codler
Copy link
Owner

codler commented Aug 5, 2012

oh, is that why. To get Developer ID I must have Mac Developer Program membership and it cost $99/year which is expensive. As a student that is too much for me =( It would be okey with one time fee, but per year is...

@mac-cain13
Copy link
Contributor Author

I get that, my company has an Mac Developer Account and can sign builds, so maybe we can figure something out to get out signed builds. At least I can sign the builds I make and put online. :)

@codler
Copy link
Owner

codler commented Aug 8, 2012

I decided to join the program :P and now I can sign.

@mac-cain13
Copy link
Contributor Author

Great to hear you've joined. :) Looking forward to a lot of signed builds!

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.

2 participants