Skip to content

Conversation

@kmkom
Copy link
Contributor

@kmkom kmkom commented May 3, 2025

For numbering filenames, added a spin box for the starting number and a spin box for the step size of the numbering in GTK ui file. Did this for both the replace and insert widgets.
Modified the Python code for the replace and insert functions accordingly.

In the replace function, ignore empty find string inputs to avoid unexpected replacement behavior.

The inject function has been modified to allow larger numbers with leading zeros (%n, %0n, %000000n, etc).

fix issue #63 and #66(partly?)

For numbering filenames, added a spin box for the starting number and a spin box for the step size of the numbering in GTK ui file.
Did this for both the replace and insert widgets.
Modified the Python code for the replace and insert functions accordingly.

In the replace function, ignore empty find string inputs to avoid unexpected replacement behavior.

The inject function has been modified to allow larger numbers with leading zeros (%n, %0n, %000000n, etc).

modified:   usr/lib/bulky/bulky.py
modified:   usr/share/bulky/bulky.ui
@kmkom
Copy link
Contributor Author

kmkom commented May 3, 2025

Tested locally here with Debian 13 and Cinnamon. It worked without any problems. However, this is my first GTK code patch. I hope it will be accepted after testing. :-)

@kmkom kmkom closed this May 3, 2025
@kmkom kmkom reopened this May 3, 2025
@kmkom
Copy link
Contributor Author

kmkom commented May 4, 2025

example:
bulky_1

@mtwebster mtwebster merged commit 6f2fa93 into linuxmint:master May 27, 2025
2 checks passed
@kmkom
Copy link
Contributor Author

kmkom commented May 28, 2025

Glad that you adopted the change. 👍 I also removed the unnecessary import of the magic module from the script. Unfortunately, I overlooked that the now unnecessary module "python3-magic" is still listed as a build dependency in the control file. It would be great if you could remove this one line from the control file as well. 😄

@mtwebster
Copy link
Member

Done, thanks
37f91c7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants