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

Added a script to to toggle between current wallpaper and live wallpaper using mpvpaper #460

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

devnchill
Copy link
Contributor

@devnchill devnchill commented Sep 26, 2024

… wallpaper

Pull Request

Description

Please read these instructions and remove unnecessary text.

  • Try to include a summary of the changes and which issue is fixed.

Used mpvpaper to apply live wallpapers . Made a custom script and added super alt 2 to toggle between current wallpaper and live wallpaper . Path for live wallpaper is in wallpapers/livewallpaper/

  • Also include relevant motivation and context (if applicable).

They look nice

  • List any dependencies that are required for this change. (e.g., packages or other PRs)

It needs mpvpaper for it . I installed it using ninja . I followed these steps

Clone

git clone --single-branch https://github.com/GhostNaN/mpvpaper

Build

cd mpvpaper
meson setup build --prefix=/usr/local
ninja -C build

Install

ninja -C build install

  • Provide a link if there is an issue related to this pull request. e.g., Fixes # (issue)
  • Please add Reviewers, Assignees, Labels, Projects, and Milestones to the PR. (if applicable)

Type of change

Please put an x in the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • [ X] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (non-breaking change; modified files are limited to the documentations)
  • Technical debt (a code change that does not fix a bug or add a feature but makes something clearer for devs)
  • Other (provide details below)

Checklist

Please put an x in the boxes that apply:

  • [X ] I have read the CONTRIBUTING document.
  • [X ] My code follows the code style of this project.
  • [ X] My commit message follows the commit guidelines.
  • My change requires a change to the documentation.
  • I want to add something in Hyprland-Dots wiki.
  • I have added tests to cover my changes.
  • [ X] I have tested my code locally and it works as expected.
  • All new and existing tests passed.

Screenshots

(if appropriate)

Additional context

Add any other context about the problem here.

@JaKooLit JaKooLit changed the base branch from main to development September 27, 2024 04:30
@JaKooLit
Copy link
Owner

mpv paper is just using wayyy too much resources.. no issues for high powered or high ram system..

but a dual core 2gb will just crawl

@devnchill
Copy link
Contributor Author

even if it's not being used ? like if you don't apply it will it still consume resources in background ?if yes then we can close it otherwise i don't see any harm in it as users can easily toggle it

@JaKooLit
Copy link
Owner

not sure... reason why I cant merge yet is I have to check and test properly. I cant now since im fully occupied with work.

I may have time to check next month

@devnchill
Copy link
Contributor Author

Sure

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