Skip to content

Add full-screen-only option#64

Merged
b3by merged 1 commit intob3by:masterfrom
matthew-e-brown:full-screen-only
Jun 22, 2019
Merged

Add full-screen-only option#64
b3by merged 1 commit intob3by:masterfrom
matthew-e-brown:full-screen-only

Conversation

@matthew-e-brown
Copy link
Copy Markdown
Contributor

Hello!

I was inspired to add this option from cmd-johnson/atom-battery-status, and I followed their code to implement this. I think it sticks with the simplicity that this package is aiming for. 😄

Hopefully I matched your programming style, I did my best to put my code in the right places. I've never developed on an :atom: package before.

@b3by b3by merged commit 3ea97f8 into b3by:master Jun 22, 2019
@b3by
Copy link
Copy Markdown
Owner

b3by commented Jun 22, 2019

Thanks for the PR! I just merged it, it looks nice indeed! If you don't mind I will just fix a couple of style things, but other than that it looks alright.

@matthew-e-brown
Copy link
Copy Markdown
Contributor Author

Go right ahead! Thanks for the merge 😄

@matthew-e-brown matthew-e-brown deleted the full-screen-only branch June 25, 2019 13:22
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