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 format_quality option in wireless #292

Merged
merged 1 commit into from Jun 29, 2018

Conversation

Stunkymonkey
Copy link
Contributor

fix for #25

Copy link
Member

@Airblader Airblader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Just some minor things. Please write a comment when you fixed it.

man/i3status.man Outdated
@@ -300,7 +300,8 @@ There also is an option "format_down". You can hide the output with

Gets the link quality, frequency and ESSID of the given wireless network
interface. You can specify different format strings for the network being
connected or not connected.
connected or not connected.The quality is padded with leading zeroes by
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be a space after the period.

man/i3status.man Outdated
@@ -300,7 +300,8 @@ There also is an option "format_down". You can hide the output with

Gets the link quality, frequency and ESSID of the given wireless network
interface. You can specify different format strings for the network being
connected or not connected.
connected or not connected.The quality is padded with leading zeroes by
default; to pad with something else use +format_quality=" "+.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should just be

[…] use +format_quality+.

@Stunkymonkey
Copy link
Contributor Author

fixed your requested changes

Copy link
Member

@Airblader Airblader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Airblader Airblader merged commit 3029663 into i3:master Jun 29, 2018
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