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

Leftwm Theme Manager Initial Commit #1

Merged
merged 6 commits into from
Feb 18, 2021
Merged

Conversation

mautamu
Copy link
Member

@mautamu mautamu commented Feb 16, 2021

Howdy,

This is part of a multi-commit PR for leftwm/leftwm, leftwm/leftwm-theme, and leftwm/leftwm-community-themes.

The new utility will allow for easier installation of themes per leftwm/leftwm#27. The currently developed commands are as follows:

SUBCOMMANDS:
    add       downloads a theme
    help      Prints this message or the help of the given subcommand(s)
    list      lists installed LeftWM themes
    new       creates a new theme
    remove    removes an installed LeftWM theme
    search    fetches a list of LeftWM themes and searches for new ones
    set       sets your current theme and restarts LeftWM
    update    fetches updates to all themes

If I want to install the Coffee theme for example, I need to run the following:

leftwm-theme update
leftwm-theme add "Coffee"
leftwm-theme set "Coffee"

I'm sure the files could be better organized, and I trust someone will help with that. A few other commands were mentioned in leftwm/leftwm#27 that I'd like to get to, including next and the one mentioned in leftwm/leftwm#191 which is leftwm-theme check and leftwm-check.

Currently, the installer does not check if a theme is valid, it operates under the assumption that only files in known.toml will make it in, locked at a particular commit, so we know they're good. That would be a good next step I think.

Best,
Mautamu

@lex148 lex148 merged commit b27762c into leftwm:master Feb 18, 2021
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