Re: [Gimp-developer] Spyrogimp plugin rewrite

2018-11-14 Thread Jehan Pagès via gimp-developer-list
Hi!

On Sun, Oct 14, 2018 at 12:47 AM Elad Shahar  wrote:

> Hi Jehan,
>
> Thanks for your feedback!
> The new version is here
>  .
>
> Here is what I have done:
>
> * I made "Esc" close the dialog (and cancel the pattern).
> * The issue with the broken icon was part of a larger issue that made the
> plugin look different than other plugins. This was resolved by using
> gimpui.py
> * I added a non-interactive API.
> * I made the dialog less tall, by grouping parameters in notebook tabs.
>
> In addition:
>
> * Using the "selection" shape now draws multiple shapes - if several paths
> were generated from the selection-to-path conversion.
> * Several new multi-sided shapes were added as fixed rings, with
> additional options.
>   These produce drawings similar to many guilloche patterns. Examples for
> the new shapes are here
> .
> * I added "long-gradient" support, that spreads across the entire pattern.
>   This was available in the previous spyrogimp.scm, and produces nice
> results which are difficult to obtain when trying to tune the gradient from
> tool settings.
> * Improved the speed of incremental drawing by using gobject.idle_add
> instead of timeouts.
>
> I'd be glad to fix any other issues.
>

So I finally reviewed.

* The background color of self.pattern_notebook is always white, which is
especially a problem with darker themes. Is it only for me? Don't you have
this issue too? I had a look and am unsure where this comes from though
(maybe it's a problem in the theme, but I have no idea).

* I had once a warning about broken undo when setting "Clone" (then I had a
warning about no clone source, but this one is normal) then canceling with
Esc.

Apart from these, it looks good here. :-)

Jehan


> If the plugin is indeed updated in the repository, could I write
> documentation for the manual?
>
> Thanks!
> Elad
>
> On Sun, Sep 16, 2018 at 10:34 PM Jehan Pagès 
> wrote:
>
>> Hi Elad,
>>
>> On Sat, Sep 15, 2018 at 4:14 PM Elad Shahar via gimp-developer-list <
>> gimp-developer-list@gnome.org> wrote:
>>
>>> Hi,
>>>
>>> Long ago, I have written a Spyrogimp plugin in scheme. The plugin is
>>> currently included in gimp (under Filters -> Render -> Spyrogimp). Now I
>>> have done a rewrite in python which I hope is a big improvement:
>>>
>>> * It provides immediate feedback, by incremental drawing to a temporary
>>> layer.
>>> * Supports using more tools to draw the pattern (e.g. stroke).
>>> * You can use a non-rectangular selection to serve as the shape of the
>>> "fixed ring". This is done by converting the selection to a path. If the
>>> path has more than one stroke, then a pattern is drawn only for one of
>>> them. ( I might improve that in the near future).
>>> * There is an additional way to specify the pattern, that is compatible
>>> with the notation in the toy kit Spirograph manuals.
>>> * Lots of tooltips
>>>
>>> If you want to try it, you can download it here:
>>> https://www.dropbox.com/s/r2t5o4n4kyvtkmi/spyro.py?dl=0
>>
>>
>> That's a cool update, and we could replace the old spyro by the new one
>> (or on 2.10 at least deprecate the old one and hide it from menus but leave
>> it alongside for the PDB API).
>> I wonder if this could not be a GEGL operation also by the way, rather
>> than a plug-in.
>>
>> Feedback is welcome.
>>>
>>
>> * Would be nice that hitting "Esc" close the dialog (and cancel the
>> pattern).
>> * On my desktop (GNOME on Fedora 28), the dialog shows a broken icon.
>> * The dialog is much too high. On my screen, part of it is out of screen
>> (the buttons in particular) so I need to use the Super key + left mouse
>> click trick to move the window. It would be nice to rearrange the buttons
>> differently or hide a scrollbar.
>> * Your new plug-in doesn't provide a non-interactive API as the old one
>> used to. I think this would be quite needed to be able to replace the old
>> plug-in correctly.
>>
>> Apart from these, that is a really cool plug-in. I would update the
>> repository with this new plug-in once these few points are fixed. :-)
>> If you answer by email, make sure to name me so that I don't miss the
>> email (I have filters allowing me to see when I am named so that I don't
>> miss mailing list emails targeted at me).
>> Thanks!
>>
>> Jehan
>>
>>
>>>
>>> Elad
>>> ___
>>> gimp-developer-list mailing list
>>> List address:gimp-developer-list@gnome.org
>>> List membership:
>>> https://mail.gnome.org/mailman/listinfo/gimp-developer-list
>>> List archives:   https://mail.gnome.org/archives/gimp-developer-list
>>>
>>
>>
>> --
>> ZeMarmot open animation film
>> http://film.zemarmot.net
>> Liberapay: https://liberapay.com/ZeMarmot/
>> Patreon: https://patreon.com/zemarmot
>> Tipeee: https://www.tipeee.com/zemarmot
>>
>

-- 
ZeMarmot open animation film
http://film.zemarmot.net
Liberapay: 

Re: [Gimp-developer] GIMP Hacking:TODO page is not updated

2018-11-14 Thread Alexandre Prokoudine via gimp-developer-list
On Wed, Nov 14, 2018 at 4:02 AM Massimo Fidanza via gimp-developer-list <
gimp-developer-list@gnome.org> wrote:

> Hi, the wiki page https://wiki.gimp.org/wiki/Hacking:TODO is not updated,
> I
> see some of the bugs are now closed. I would like to update the page and
> report the current situation, how can I help?
>

Hi, thanks for reminding, totally overlooked this. Fixed.

Alex
___
gimp-developer-list mailing list
List address:gimp-developer-list@gnome.org
List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list
List archives:   https://mail.gnome.org/archives/gimp-developer-list


Re: [Gimp-developer] GIMP Hacking:TODO page is not updated

2018-11-14 Thread JLuc

Le 14/11/2018 à 02:02, Massimo Fidanza via gimp-developer-list a écrit :

Hi, the wiki page https://wiki.gimp.org/wiki/Hacking:TODO is not updated,


Lol.
;-)

___
gimp-developer-list mailing list
List address:gimp-developer-list@gnome.org
List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list
List archives:   https://mail.gnome.org/archives/gimp-developer-list