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

LXDE Support? #17

Open
Fabioamd87 opened this issue Apr 3, 2017 · 33 comments
Open

LXDE Support? #17

Fabioamd87 opened this issue Apr 3, 2017 · 33 comments

Comments

@Fabioamd87
Copy link

Fabioamd87 commented Apr 3, 2017

The script says:

fabio@a514:~$ bingwallpaper 
Pinging Bing API...
Bing Image of the day: http://www.bing.com/az/hprichbg/rb/DivingGondola_EN-US11480704756_1920x1200.jpg
Saving image to /home/fabio/Pictures/Bing/DivingGondola_EN-US11480704756_1920x1200.jpg
New wallpaper set successfully for LXDE.

but the wallpaper didn't get changed

@Fabioamd87
Copy link
Author

solved by editing

.config/lxsession/Lubuntu/autostart

(that was empty) and adding:

@lxpanel --profile Lubuntu
@pcmanfm --desktop --profile Lubuntu
@xscreensaver -no-splash

I think is the second line was necessary

@Fabioamd87
Copy link
Author

After the update of tomorrow of Lubuntu 17.04 it seems to not work anymore.
The output isn't changed but the wallpaper doesn't change, I've checked and the options are still there in the autostart file

@dzmanto
Copy link

dzmanto commented Apr 20, 2017 via email

@Fabioamd87
Copy link
Author

fabio@a514:~/Pictures/Bing$ pcmanfm -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg 
fabio@a514:~/Pictures/Bing$ pcmanfm-qt -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg 
** Message: x-terminal-emulator has very limited support, consider choose another terminal
isPrimaryInstance
fabio@a514:~/Pictures/Bing$ 

I'm using lxde and not lxqt BTW with the first command I don't get results. with the second I get the error in terminal.

Thanks

@dzmanto
Copy link

dzmanto commented Apr 20, 2017 via email

@Fabioamd87
Copy link
Author

Hi Daniela,
With both commands the wallpaper remains the same.

@dzmanto
Copy link

dzmanto commented Apr 21, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented Apr 21, 2017

I am sorry Daniela but I think I didn't understand correctly your first comment.

using

pcmanfm -w "/path/to/your/background"

without quotes works. Relative path in your last comment not.

but I still have problems with using pcmanfm-qt, that gives me the error that I've posted in my last message "x-terminal-emulator has very limited support"
it's the same errore that I get if I start "pcmanfm-qt" from terminal, it's not related with the script.

if the script would use pcmanfm instead of pcmanfm-qt it would be ok I suppose

@dzmanto
Copy link

dzmanto commented Apr 21, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented Apr 24, 2017

> fabio@a514:~/bin/linux-bing-wallpaper$ chmod +x bing_wallpaper.sh 
> fabio@a514:~/bin/linux-bing-wallpaper$ sudo ./bing_wallpaper.sh 
> [sudo] password for fabio: 
> ./bing_wallpaper.sh: line 231: feh: command not found

but also after installing feh, the script, given as sudo, doesn't give any output and the wallpaper doesn't change.

also given as normal user.

@dzmanto
Copy link

dzmanto commented Apr 24, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented Apr 24, 2017

fabio@a514:~/bin/linux-bing-wallpaper$ echo ${XDG_CURRENT_DESKTOP}
LXDE
fabio@a514:~/bin/linux-bing-wallpaper$

thanks again

@dzmanto
Copy link

dzmanto commented Apr 24, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented Apr 24, 2017

Hi Daniela, unfortunately still no result but thanks for following my problem.
what do you mean with "enclosed"? I started the script as root after installed feh and it doesn't work with no errors now.
as I said using pcmanfm instead of pcmanfm-qt works

@dzmanto
Copy link

dzmanto commented Apr 25, 2017 via email

@Fabioamd87
Copy link
Author

Hi Daniela,
I am sorry but I cannot see the ATTACHMENT.
Im reading the conversation on github support website.

@dzmanto
Copy link

dzmanto commented Apr 25, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented Apr 26, 2017

Hi daniela,
The script you've posted works correctly.

PS: I've noticed however, that the script never ends, also after changed the wallpaper, it would be nice to change the wallpaper with one command line.

@dzmanto
Copy link

dzmanto commented Apr 26, 2017 via email

@Fabioamd87
Copy link
Author

Hi Daniela,
Let me understand, it is a different repo?

@dzmanto
Copy link

dzmanto commented Apr 27, 2017 via email

@Fabioamd87
Copy link
Author

Thanks Daniela,
As a suggest, do I need to use your repo or I can wait the main repo to be updetd?
I saw that last updates on the main repo are months ago.

@dzmanto
Copy link

dzmanto commented Apr 27, 2017 via email

@Fabioamd87
Copy link
Author

Fabioamd87 commented May 2, 2017

Hi Daniela,
I have noticed that the image file are no longer saved in the Bing folder, do you confirm this?

thanks,
Fabio

@dzmanto
Copy link

dzmanto commented May 3, 2017 via email

@Fabioamd87
Copy link
Author

Hi Daniela,
I think this is a correct way of thinking, but it is also nice to have save this beautiful wallpaper somewhere.
What do you think about that? Maybe I can work on that.

@dzmanto
Copy link

dzmanto commented May 3, 2017 via email

ARR8 added a commit to ARR8/linux-bing-wallpaper that referenced this issue Jun 4, 2018
ARR8 added a commit to ARR8/linux-bing-wallpaper that referenced this issue Jun 4, 2018
marguerite pushed a commit that referenced this issue Jun 4, 2018
@bugz8unny69
Copy link

Hi @dzmanto, I checked out your forked repo, 1. I see you're using ba(sh) instead of golang and 2. unfortunately you don't have issues enable for that repo. Should we be reporting issues on this ticket?

@dzmanto
Copy link

dzmanto commented Oct 25, 2018

Hi @LHorace, how are issues enabled for this repository? Also, you are more than welcome to report issues. Cheers, Dani

@bugz8unny69
Copy link

bugz8unny69 commented Oct 25, 2018

Hi @dzmanto, check out https://help.github.com/articles/disabling-issues/, simply do the reverse?

You script in it's current implementation right now, attempts to download whatever image BING offers, than later resize image to higher resolution, however unfortunately, convert here is returning a black image on convert -resize. I would rather prefer attempting to download the highest resolution image possible, than fallback to whatever image BING gives, than resize. This would avoid image quality lost by up-scaling a image in some cases?

@dzmanto
Copy link

dzmanto commented Oct 29, 2018 via email

@dzmanto
Copy link

dzmanto commented Oct 29, 2018 via email

@bugz8unny69
Copy link

Sure, when I get time to do so! It's just rearranging your code, not to much work!

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

No branches or pull requests

3 participants