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

Use single rendering script for all variations #386

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hsbasu
Copy link

@hsbasu hsbasu commented May 9, 2022

  • uses single render-assets.sh for all variations
  • single rendering script takes care of all assets
    used in gtk variations and xfwm variations
  • reduces repo size and better automation
  • No need to include .png files with repo as
    they can be generated during building deb packaging
  • better rendering of assets for new themes like gtk-4.0
  • original rendering scripts has been kept back but can be removed

@JosephMcc
Copy link
Contributor

Thanks for the effort. I want to hold off a bit on merging this. Currently working on some updating to the themes and repos. It will likely change some of the assets that are being used and some can likely be shared between gtk3 and 4 versions to cut down on the moving parts even more.

@hsbasu
Copy link
Author

hsbasu commented May 10, 2022

So, should I keep this PR open? Or close it and create a new PR with updated master?

@JosephMcc
Copy link
Contributor

You can keep it open. It may still end up working as is. I'll take a look back at it later on.

@hsbasu
Copy link
Author

hsbasu commented May 11, 2022

Ok. Thanks for the hard work.

- uses single render-assets.sh for all variations
- single rendering script takes care of all assets
  used in gtk variations and xfwm variations
- reduces repo size and better automation
- No need to include .png files with repo as
  they can be generated during building deb packaging
- better rendering of assets for new themes like gtk-4.0
- original rendering scripts has been kept back but can be removed
@hsbasu hsbasu force-pushed the use-single-rendering-script branch from f973e82 to c03d709 Compare May 14, 2022 10:19
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