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

module: add more module/global options #1481

Closed
wants to merge 4 commits into from
Closed

Conversation

lasers
Copy link
Contributor

@lasers lasers commented Sep 2, 2018

Addresses #1078.

@lasers lasers changed the title Module new options module: support more colors, support more module/global options Sep 2, 2018
@lasers lasers changed the title module: support more colors, support more module/global options module: add more colors and module/global options Sep 2, 2018
@lasers lasers added the enhancement 😍 I have created an enhancement label Sep 2, 2018
@lasers lasers force-pushed the module-new-options branch 3 times, most recently from 98e83f9 to 5e176d0 Compare September 2, 2018 11:23
@tobes
Copy link
Contributor

tobes commented Sep 2, 2018

hey @laser can you split this

you have

  • unrelated documentation changes
  • changes to default settings
  • changes to module.py current implementation
  • added features

These all need to be done separately

@lasers lasers changed the title module: add more colors and module/global options module: add more module/global options Sep 2, 2018
@lasers
Copy link
Contributor Author

lasers commented Sep 2, 2018

@tobes You got it, pal. 👍 I wait for other PRs to be merged first before I rebase this.

Copy link
Contributor

@tobes tobes left a comment

Choose a reason for hiding this comment

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

These are just doc fixes so we can merge them on their own

@@ -71,8 +71,7 @@ overridden by also defining in the individual module.

Global options:

- ``nagbar_font``. It will be used as an argument to
``i3-nagbar -f``, thus setting its font.
``nagbar_font``. Specify a font for ``i3-nagbar -f <font>``.
Copy link
Contributor

Choose a reason for hiding this comment

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

this is unrelated

@@ -93,7 +92,7 @@ Store cache in $XDG_CACHE_HOME or ~/.cache

# default behavior
py3status {
storage = 'py3status_cache.data'
storage = 'py3status_cache.data'
Copy link
Contributor

Choose a reason for hiding this comment

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

this is unrelated

@@ -294,7 +330,7 @@ You can:
format = "MPD: {state} {artist} {title}"
}

- Unknown placeholders act as if they were static text and placeholders that are empty or None will be removed.
- Unknown placeholders act as if they were static text and placeholders that are empty or None will be removed.
Copy link
Contributor

Choose a reason for hiding this comment

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

this is unrelated

@ultrabug
Copy link
Owner

ping @lasers closing this one since there's too much in it, would be cool to propose only the alpha thing on a simple PR!

cheers

@ultrabug ultrabug closed this Dec 23, 2018
@lasers
Copy link
Contributor Author

lasers commented Dec 23, 2018

I don't have alpha thing working. This PR would need to be fixed either way after merging #1486.

#1486 works okay. I made temp get_color function instead of importing Py3's get_color().

@lasers lasers deleted the module-new-options branch December 23, 2018 18:17
@lasers lasers restored the module-new-options branch December 23, 2018 18:17
@lasers lasers reopened this Dec 30, 2018
@ultrabug
Copy link
Owner

@lasers if you make it a simple and one-thing-at-a-time PR, why not

this PR is not that, closing, open a new one if you wish

@ultrabug ultrabug closed this Dec 30, 2018
@lasers lasers deleted the module-new-options branch December 30, 2018 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 😍 I have created an enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants