After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 88633 - Border Selection Without feathering
Border Selection Without feathering
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Tools
1.x
Other All
: Low enhancement
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2002-07-19 17:46 UTC by rwf
Modified: 2006-06-05 18:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add "unfeathered" option to Selection Border (9.34 KB, patch)
2006-06-04 16:11 UTC, saulgoode
needs-work Details | Review

Description rwf 2002-07-19 17:46:02 UTC
Hello,

It would be very nice to be able to border a selection without having the 
resulting selection feathered! The GIMP Users Manual simply 
states: "Selections created using Border selection are feathered" so I'm 
assuming there's no current option to disable this behavior.

Thanks for your time!

-Fredric Echols

rwf@gpcom.net
Comment 1 Sven Neumann 2003-01-29 13:19:10 UTC
Adding a toggle to make this behaviour optional sounds like a good
idea for 1.3.
Comment 2 Dave Neary 2003-07-26 10:33:14 UTC
Bumping a bunch of enhancement requests which will not be done by the feature
freeze to Future. 

Dave.
Comment 3 Sven Neumann 2005-07-26 13:01:50 UTC
This can already be easily achieved by calling Select->Sharpen after bordering
the selection. Does it really make sense to clutter the UI with another check
button?
Comment 4 Carol 2006-04-26 04:19:10 UTC
This is such an old enhancement request.

If adding the toggle is too much clutter to the already extremely simple UI and the knowledge that Select -> Sharpen is expected, can this report be reassigned to   the Help component.

I was actually surprised to see that this is part of the gimp internals.  

After a few months of reflection and dealing with GIMP Usability, perhaps adding the toggle is not such an unreasonable suggestion.
Comment 5 Carol 2006-04-26 05:47:22 UTC
This bug report is similar to 86279.

I am adding gnome-love to the keywords.
Comment 6 saulgoode 2006-06-04 16:11:02 UTC
Created attachment 66734 [details] [review]
Patch to add "unfeathered" option to Selection Border

This patch adds a checkbox to the <image>/Selection/Border command which toggles feathering on and off. This patch adds a boolean "feather_border" argument to the relevant core functions but hardcodes this argument to TRUE for the PDB functions so as not to alter the plugin interface.

NOTE: the result of an unfeathered operation is not precisely the same as if a "Selection Sharpen" were performed after a feathered "Selection Border"; despite the PDB notes, Selection Sharpen seems to use a threshold of "127" and not "0". I haven't had time to see if there is an existing report for this disparity.

The patch is relative to CVS head.
Comment 7 Sven Neumann 2006-06-04 17:17:35 UTC
Sorry, but the patch doesn't look as if it was created against the HEAD branch from CVS.  In general it looks fine, despite a number of coding style issues and the fact that the *_cmds.c files in the app/pdb directory are generated code.
Comment 8 Michael Natterer 2006-06-05 15:25:00 UTC
The patch is ok anyway, will look into applying it.
Comment 9 Michael Natterer 2006-06-05 18:34:18 UTC
Fixed in CVS:

2006-06-05  Michael Natterer  <mitch@gimp.org>

	Applied slightly modified patch from saulgoode which allows to
	border a selection without feathering. Fixes bug #88633.

	* app/paint-funcs/paint-funcs.[ch] (border_region): added boolean
	"feather_border" parameter.

	* app/core/gimpchannel.[ch]: GimpChannel::border(): ditto.

	* app/core/gimpselection.c: changed accordingly.

	* app/actions/select-commands.c: added a toggle to the border
	dialog and pass it to gimp_channel_border().

	* tools/pdbgen/pdb/selection.pdb: pass TRUE here.

	* app/pdb/selection_cmds.c: regenerated.