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

resize_png can now create multiple icons at once #466

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

Htylol
Copy link
Contributor

@Htylol Htylol commented Jan 19, 2025

  1. resize_png теперь одной командой может создать 48, 128 и т.п. размеры. Для чего это нужно, когда к примеру из проводника запустить какой-нибудь .exe который находится на временно примонтированом диске, добавить на него ярлык, запустить, потом всё закрыть не заходя в главное меню, перезапустить систему без этого диска, в главном меню не будет 48 иконки, так он создает этого пра только 128., 48 и 128 создаёт только из главного меню.
  2. добавил fix_icon_name_png, который фиксит строку Icon=путь и название до png, если в нём есть разного рода символы, из-за них yad работает некорректно. Сделано это для того, чтобы из меню запуска игры, когда он создаёт png, то создавал png правильные, без этих символов.
  3. ну и других местах внёс изменения, это для иконки при создании ярлыков и изменении и при запуске самого приложения, name_desktop_png используется при создании .desktop файла, чтобы он сразу создавал без правильное название у png без символов , fix_icon_name_png нужен для обратной совместимости, когда уже .destkop файл был создан. Есть вариант немного упростить пункт под номером 3, но тогда PORTPROTON_NAME принудительно в ppdb менять нужно, а он создаётся один раз при использовании exiftool и в дальнейшем статичен .

@Htylol Htylol force-pushed the update_resize_png branch from 2ff8516 to cb8f27e Compare January 19, 2025 09:55
@Castro-Fidel Castro-Fidel merged commit 2332b93 into Castro-Fidel:devel Feb 4, 2025
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