Filter editor: only show relevant light source parameters for lighting filter primitives (rev >= 10957)

Bug #1095491 reported by v1nce
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Triaged
Medium
Unassigned

Bug Description

when switching from 0.48.3 to 0.48-devel-12005-37 I see some "regression" in lighting filter

now the parameters for all type of light source(distant,point,spot) are visible at once.

this is bad. it eats so much space while there is so little for filter lists which is the core of the dialog.

can this be reverted to 0.48.3 ? or can one replace the light source dropdown with 3 tabs ?

by the way if one source of lighti (distant ?) could be selected by default it would be great (who wants to create a feDiffuseLighting without a lightsource ?)

Revision history for this message
v1nce (vincent-pennec) wrote :

please someone rewrites title as "[wish] light source parameters eat too much space (0.48.3 was better)"

su_v (suv-lp)
tags: added: filters-svg ui
Revision history for this message
su_v (suv-lp) wrote :

> now the parameters for all type of light source(distant,point,spot) are visible at once.

Not reproduced with Inkscape 0.48.4 on OS X 10.7.4,
reproduced with Inkscape 0.48+devel r12006

Each time e.g. the user switches between different filter primitives of the current filter, and reselects one of the employed lighting filters, all parameters instead of only those relevant to the chosen light source are displayed in the filter editor dialog.

Based on archived builds, this regression seems to have started with r10957 or 10958 (not reproduced with r10956, reproduced with r10958):
- Revision 10957: "Lots of gtkmm deprecation fixes"
  <http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/10957>
- Revision 10958: "GSEAL fixes"
  <http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/10958>

tags: added: regression
Revision history for this message
su_v (suv-lp) wrote :

@Alex - maybe you could take a closer look what is happening here?

Changed in inkscape:
milestone: none → 0.49
status: New → Confirmed
Revision history for this message
su_v (suv-lp) wrote :

Also reproduced with trunk r12006 on Ubuntu 12.10 (64bit, VM).

summary: - don't list all type lighting filters
+ Filter editor: don't list all light source parameters of lighting filter
+ primitives
summary: - Filter editor: don't list all light source parameters of lighting filter
- primitives
+ Filter editor: only show relevant light source parameters for lighting
+ filter primitives
su_v (suv-lp)
Changed in inkscape:
importance: Undecided → Medium
status: Confirmed → Triaged
su_v (suv-lp)
summary: Filter editor: only show relevant light source parameters for lighting
- filter primitives
+ filter primitives (rev >= 10957)
Revision history for this message
jazzynico (jazzynico) wrote :

> the parameters for all type of light source(distant,point,spot) are visible at once.

Tested again with revision 13220 on Crunchbang and Windows XP.
Selecting a light source in the list now shows its set of parameters, and not everything as in the screenshot from comment #2. But unfortunately, all the parameters are still visible when no light source is selected.

su_v (suv-lp)
Changed in inkscape:
milestone: 0.91 → 0.92
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.