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 359833 - Lighting Effects input fields disabled
Lighting Effects input fields disabled
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
2.2.x
Other All
: Normal minor
: 2.2
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2006-10-05 09:34 UTC by rhi
Modified: 2008-01-15 13:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Enable/disable fields when changing light type (2.92 KB, patch)
2007-04-13 10:53 UTC, Aurimas Juška
committed Details | Review

Description rhi 2006-10-05 09:34:15 UTC
Please describe the problem:
When applying the lighting effects filter, input fields for "light" / "light settings" / "Light 2" are disabled wrongly

Steps to reproduce:
1. Create a new image
2. Filters / Light Effects / Lighting Effects
3. Choose tab "Light"
4. "Light 1" is selected => set (move) it
5. Select "Light 2"
6. Set type, e.g. "Point"
7. "Position"/"Directon" can NOT be set

Workaround:
8. Select "Light 1", then select "Light 2" again
7. "Position"/"Directon" CAN be set

Actual results:
The position/direction input fields are not activated after step 6

Expected results:
Position/direction input fields should be activated after step 6

Does this happen every time?
yes

Other information:
Comment 1 Aurimas Juška 2007-04-13 10:53:17 UTC
Created attachment 86281 [details] [review]
Enable/disable fields when changing light type
Comment 2 Sven Neumann 2007-04-16 06:16:49 UTC
Thanks a lot for the patch.

2007-04-16  Sven Neumann  <sven@gimp.org>

	* plug-ins/Lighting/lighting_ui.c: applied patch from Aurimas
	Juška that fixes sensitivity of input fields related to the light
	type (bug #359833).
Comment 3 Sven Neumann 2007-04-16 06:18:41 UTC
I have also merged this change to the release branch.