Bug fix for Bug 127571 - new feature: split the image
Boudewijn Rempt
boud at valdyas.org
Tue Mar 22 10:03:59 CET 2011
On Monday 21 March 2011 Mar, srikanth tiyyagura wrote:
> Hi to all,
>
> This is bug fix for Bug 127571 - new feature: split the image .
> I used the code of some other files and prepared this one. Check out
> once and correct me if there are any mistakes.
>
> ImageSplit is a widget , which is used to split the image based on
> vertical split lines and also horizontal lines, which are provided by
> the user. These splitted images are saved without effecting the main
> image.
Cool stuff! Works like a charm and it looks pretty clean already.
A couple of remarks:
* replace my copyright with your copyright and update the year. I know you started by copying files from another plugin, but that was just as a template. This is your code!
* set a shortcut on the action, or create a new action to apply the saved split with the keyboard only, without opening the dialog. Maciej will be very glad withthat.
* use the original filename + N as name, not imagesplit_N.kra
* maybe ask the user for a filetype, like png or jpg
* the dialog needs another layout to show up correctly, here (because I use a large font size), part of the dialog is cut off.
Do you already have commit access or shall I commit this version so you can continue working on it?
--
Boudewijn Rempt | http://www.valdyas.org, http://www.krita.org
More information about the kimageshop
mailing list