GNOME Bugzilla – Bug 582755
Addition of golden spiral to rectangle tool guides
Last modified: 2018-05-24 12:34:06 UTC
I have made a patch that add a golden spiral (fibonacci spiral) to the rectagle tools guides. I mainly use it for cropping photographs. There is some explanation of the use of this guide can be found in the following links. http://www.powerretouche.com/Divine_proportion_tutorial.htm#tutorial http://cwlawrencephoto.blogspot.com/2006/04/golden-section-in-photo-composition.html http://www.fabiovisentin.com/blog/45.ashx I am submitting a patch relative to the 2.6.6. I hope it to be of interest to other gimp users. I will be happy to add alterations if needed. Boris
Created attachment 134699 [details] [review] patch for the enhancement
Thanks! Let's consider adding this for 2.8.
This patch implements several features at the same time. It would be nice if you could provide a patch that only adds the golden spiral. The other changes can be discussed separately.
(In reply to comment #3) > This patch implements several features at the same time. It would be nice if > you could provide a patch that only adds the golden spiral. The other changes > can be discussed separately. > I didn't inted to add more then one feature... I added the spiral with the posibility to flip and rotate the spiral. There are 8 possible spiral positions. There is no point to add the spiral in one direction only.... Thanks, Boris
Well, the tool options are already cluttered so much. We are certainly not going to add more controls only to be able to flip and rotate a guide. You will have to come up with a solution that needs less screen estate. Also the patch has coding style issues and it won't apply against the current development tree as we already included another type of guides (see bug #566443).
(In reply to comment #5) > Well, the tool options are already cluttered so much. We are certainly not > going to add more controls only to be able to flip and rotate a guide. You will > have to come up with a solution that needs less screen estate. > > Also the patch has coding style issues and it won't apply against the current > development tree as we already included another type of guides (see bug > #566443). > O.K. I will try to come up with an other proposal that will also cleanly patch against the current developement. I was not aware of coding style issues. I'll try to discover them and correct them. Do you think that the aproach of adding to the gides combo two possibilities - vertical golden spirals and horizontal golden spirals, each of them drawing four symmetrical spiral guides on the screen would be acceptable? Thanks, Boris
Created attachment 134749 [details] [review] updated patch This is an improved solution in the lines of my previous post. It works against rev. 28270 of the trunk hopefully has no coding style issues and uses no extra controls. I am prepared to improve it farther if this is not yet acceptable. Boris
I'd just like to point out that we are using git now: http://git.gnome.org/cgit/gimp/
Created attachment 134758 [details] [review] git generated patch
(In reply to comment #8) > I'd just like to point out that we are using git now: > http://git.gnome.org/cgit/gimp/ > Thanks Martin, I had no Idea. The patch applies cleanly to the git trunk. But I attached a patch generated by git anyway. Boris
Created attachment 134766 [details] [review] cleaned up version of this patch
I find the four spirals drawn at the same time rather confusing. Also the combo box becomes very wide due to the length of the added items. Perhaps we can get a good idea from the UI department how this could be solved better? In the meantime I have attached a cleaned up version of your patch. I tried to improve the readability a little and removed the whitespace errors your patch introduced.
(In reply to comment #12) > I find the four spirals drawn at the same time rather confusing. Also the combo > box becomes very wide due to the length of the added items. Perhaps we can get > a good idea from the UI department how this could be solved better? > > In the meantime I have attached a cleaned up version of your patch. I tried to > improve the readability a little and removed the whitespace errors your patch > introduced. > Thanks. I see that the constraints in developing a program like the GIMP are more numerous then I suspected ;-). I agree that the 4 spirals are somewhat confusing at first. The only other idea I have is to add the flip check boxes conditionaly so that they will take presious screen estate only when you select the specific guides in the combo, but I am not sure whether this is acceptable. In any case I'll be happy to implement any suggestion that you'll come up with. And about the white space errors is there any automatic tool that flags these and other coding style issues? Boris
You can certainly configure your editor to show trailing white-space.
The patches themselves look fine to me.
Interesting from original author has faded away, moving off milestones.
(In reply to comment #16) > Interesting from original author has faded away, moving off milestones. I am still very interested I Just don't see a way acceptable by Sven to incorporate this fiture in the current gui. My sujestions were not accepted and there were no other sujestions. So I see no way to proceed.
Well, what this patch is waiting for for more than year now is for someone to take the discussion to the gimp-developer mailing-list and ask for advice from the UI designers.
(In reply to comment #18) > Well, what this patch is waiting for for more than year now is for someone to > take the discussion to the gimp-developer mailing-list and ask for advice from > the UI designers. Now I understand. I hade no idea that by "the UI department" you ment the UI designers at gimp-developer. Well, Sven you all developed a wonderfull software that I enjoy using very much. But it seems to me that the team could use some more patience for the first time would-be contributors doing their first steps in trying to help.
Comment on attachment 134766 [details] [review] cleaned up version of this patch per comment 12
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/gimp/issues/303.