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

Add Desktop Dimmer #425

Merged
merged 1 commit into from
Jan 30, 2017
Merged

Conversation

sidneys
Copy link
Contributor

@sidneys sidneys commented Jan 29, 2017

Added Desktop Dimmer, an open-source approach for controlling desktop brightness.

Reason:
It's been out there for a while ('soft-launched') and has so far proven to be of assistance to its small developer-heavy user base. Additional exposure would improve roadmaps' feature prioritization.

Link:
https://github.com/sidneys/desktop-dimmer

@sindresorhus
Copy link
Owner

I tried the app, but didn't work (macOS):

screen shot 2017-01-30 at 02 45 24

Your app is also needlessly large. Your app source itself (without Electron) is 76 MB. Pretty sure you're forgetting to exclude some development dependencies.

@sidneys
Copy link
Contributor Author

sidneys commented Jan 29, 2017

@sindresorhus
Thanks for your time reviewing - we just released a new version which addresses this issue:
https://github.com/sidneys/desktop-dimmer/releases/tag/v2.2.0

Looking forward to your opinion.

Your comment regarding file size seems to be on point - a quick first analysis has shown that the reason are bundled compiler libraries. This will be addressed asap, but as it requires a rework of the developer scripts and associated build and packaging workflows, it'll be a major release.

@sindresorhus sindresorhus merged commit 55f9a7c into sindresorhus:master Jan 30, 2017
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