leftwm / leftwm-community-themes

A collection of themes built by the community
BSD 3-Clause "New" or "Revised" License
173 stars 23 forks source link

Anyone having problems with Blue Coffee and other themes? #21

Open ghost opened 3 years ago

ghost commented 3 years ago

I've recently installed Arch on my new laptop. When I tried to install my theme through leftwm-theme, it turned out to be broken until I swapped files with the coffee.

Theme works normally when downloaded directly from github, or when I swap files with other working themes (also apply to other themes made by community, not the stock ones)

I'll be switching from Arco to mainline Arch and try that again. Just want to ask if other people have the same problem

ghost commented 3 years ago

Since I don't want to create another topic for something this small: What is a keybind for pulse audio (volume up/down/mute)?

mautamu commented 3 years ago

To answer the second question first:

Since I don't want to create another topic for something this small: What is a keybind for pulse audio (volume up/down/mute)?

https://github.com/leftwm/leftwm/blob/003c9cba8f27b61ec2dba6b051c32a4b583baed1/src/utils/xkeysym_lookup.rs#L1047

If you want the volume up/down on the keyboard to work, it’s the key linked above. Just make sure to give it an empty modifier in your configuration and a value of (pulse audio command).

mautamu commented 3 years ago

As for the theme itself:

Is there something missing when installing from leftwm-theme?

ghost commented 3 years ago

As for the theme itself:

Is there something missing when installing from leftwm-theme?

No, everything is place. I switched the files with Coffee and everything worked fine. I'll do tests tomorrow and see what happens

ghost commented 3 years ago

Same issue on fresh Arch. Can you install Blue Coffee and see if it works on your side?

mautamu commented 3 years ago

Only able to reproduce with Blue Coffee theme. Error is on line 4 of up script. Additional errors on down script.

taylor85345 commented 3 years ago

edit: This doesn't seem to be the issue here. I don't see any newer commits that would fix the issue in question

leftwm-theme clones your theme from the Blue Coffee submodule in the leftwm-community-themes repository, which is a branch of your repo reflecting it's state at the time you submitted the original pull request. Any changes made to Main after your initial submission will not be reflected by a leftwm-theme install until you update the Blue Coffee submodule in leftwm-community-themes, which may be why the theme is installing in a broken state

taylor85345 commented 3 years ago

Also, it looks like your up script isn't downloading with execute permissions. chmod +x up fixed the issue for me and allowed the theme to load normally

ghost commented 3 years ago

Sorry for late reply, but my HDD died

Also, it looks like your up script isn't downloading with execute permissions. chmod +x up fixed the issue for me and allowed the theme to load normally

Thanks for the answer, I was told that leftwm-theme automatically sync with main branch. Can you tell me more about theup script issue and how to update submodule?

taylor85345 commented 3 years ago

You might actually be right. The submodule in the repository is always a little behind, but leftwm-theme install actually seems to pull from main, though upgrade doesn't seem to do anything.

To start with, you might just try adding execute permissions to the up script and pushing the change back to your github.

If you do need to update the submodule, which I'm now less confident about, you'll need to fork the leftwm-community-themes and clone it like you did when you originally submitted (see the wiki).Once you've cloned the repo, do the following:

git submodule update --init 'Blue Coffee' cd 'Blue Coffee git checkout main git pull git commit -a

You may also want to update the version number in known.toml for your theme, but not sure if that's necessary. From there just git push the changes back to github and submit a pull request (again, per the wiki).

mautamu commented 3 years ago

though upgrade doesn't seem to do anything.

Ah, I will file an issue over on leftwm/leftwm-theme about this. It is UB to do that one would suppose. Not sure how best to handle it.

ghost commented 3 years ago

You might actually be right. The submodule in the repository is always a little behind, but leftwm-theme install actually seems to pull from main, though upgrade doesn't seem to do anything.

To start with, you might just try adding execute permissions to the up script and pushing the change back to your github.

If you do need to update the submodule, which I'm now less confident about, you'll need to fork the leftwm-community-themes and clone it like you did when you originally submitted (see the wiki).Once you've cloned the repo, do the following:

git submodule update --init 'Blue Coffee' cd 'Blue Coffee git checkout main git pull git commit -a

You may also want to update the version number in known.toml for your theme, but not sure if that's necessary. From there just git push the changes back to github and submit a pull request (again, per the wiki).

Do you know what exactly to add to the up script? I simply forked the Coffee and the only thing changed in up script was using feh for background (which I borrowed from Dracula Rounded)

taylor85345 commented 3 years ago

Don't add anything to it. Just run chmod +x up from the terminal and that will add the permissions. If you're uploading files to github through a browser, that probably doesn't preserve permissions, so you'll want to git push the changes back from the terminal.

cd $HOME/leftwm/themes/Blue Coffee chmod +x up git commit up (you'll add a commit comment to the file that comes up) git push (You'll enter your github username and password)

mautamu commented 3 years ago

The issue that leftwm-theme needs to fix is to tell Git to download as blue_coffee / Blue_Coffee, and/or to go and upstream a fix for spaces in all themes with a space in the name.

Former solution blocked on leftwm/leftwm-theme#9.

VentGrey commented 1 year ago

Blue Coffee is 404'd

VuiMuich commented 1 year ago

I guess we should revert back from submodules to directly committed themes. Maybe in addition simplify how to add additional repo sources to leftwm-theme?