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

After set new wallpaper with pcmanfm-qt, wallpaper will not update without trigger #935

Closed
ghost opened this issue Apr 11, 2019 · 5 comments · Fixed by #936
Closed

After set new wallpaper with pcmanfm-qt, wallpaper will not update without trigger #935

ghost opened this issue Apr 11, 2019 · 5 comments · Fixed by #936
Labels
Projects

Comments

@ghost
Copy link

ghost commented Apr 11, 2019

Hi.
I use the dynamic wallpaper (xplanetFX), which use the external script to update wallpaper.
I decide to test updating wallpaper with terminal:
pcmanfm-qt --set-wallpaper=/home/user/picture.jpg --wallpaper-mode=fit

Expected Behavior

Wallpaper should update immediately.

Current Behavior

When cursor remain over any window region, wallpaper will not update.
Wallpaper will update after mouse cursor move over free (unoccupied with any window) desktop space.

Possible Solution
Steps to Reproduce (for bugs)
  1. Use any terminal. Set the mouse cursor under any window region;
  2. Use command: pcmanfm-qt --set-wallpaper=FILE --wallpaper-mode=MODE
  3. Do not move the mouse cursor (wallpaper will not update)
  4. Move the mouse cursor over free (unoccupied with any window) desktop space: wallpaper will update.
Context
System Information
  • Distribution & Version: Debian Buster (testing)
  • Kernel: 4.19.04 (amd64)
  • Qt Version: 5.11
  • libfm-qt Version: 0.14.1-1
  • libqtxdg Version:
  • lxqt-build-tools Version: 0.6.0-2
  • Package version:

Additional info:
Window manger: openbox or xfwm.

P.S. sorry for my bad English

@tsujan tsujan added the bug label Apr 11, 2019
tsujan added a commit that referenced this issue Apr 11, 2019
In other words, don't wait for desktop to be updated by itself.

Fixes #935
@tsujan
Copy link
Member

tsujan commented Apr 11, 2019

The problem was elementary; its fix is here: #936

BTW, you don't need any external script to update wallpaper. pcmanfm-qt supports slide shows. But, of course, external scripts work too.

@ghost
Copy link
Author

ghost commented Apr 11, 2019

Thanks for quick fixing this issue!

@ghost
Copy link
Author

ghost commented Apr 11, 2019

Have I to close this issue?

@tsujan
Copy link
Member

tsujan commented Apr 11, 2019

No. It'll be closed automatically when its corresponding fix is merged into the source.

@ghost
Copy link
Author

ghost commented Apr 11, 2019

Copy that.

@agaida agaida added this to High Prio in Issues Apr 13, 2019
Issues automation moved this from High Prio to Closed Apr 13, 2019
agaida pushed a commit that referenced this issue Apr 13, 2019
In other words, don't wait for desktop to be updated by itself.

Fixes #935
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Issues
  
Closed
Development

Successfully merging a pull request may close this issue.

1 participant