brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

magnifier: Using properly 'color-saturation'

Browse Source 
As description of the setting says, color-saturation ranges from
0.0 (grayscale) to 1.0 (full color), but the real outcome was the
opposite. The reason is that clutter provides a desaturation effect,
and color-saturation was passed directly to that effect. This patch
renames the effect and compute the desaturation value.
This commit is contained in:
Alejandro Piñeiro 2012-07-24 18:59:47 +02:00
parent ec78dd60fc
commit 7431c4d611
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
js/ui/magnifier.js
View File

@ -1641,14 +1641,14 @@ const MagShaderEffects = new Lang.Class({
_init: function(uiGroupClone) {
this._inverse = new Shell.InvertLightnessEffect();
this._brightnessContrast = new Clutter.BrightnessContrastEffect();
this._colorSaturation = new Clutter.DesaturateEffect();
this._colorDesaturation = new Clutter.DesaturateEffect();
this._inverse.set_enabled(false);
this._brightnessContrast.set_enabled(false);
this._magView = uiGroupClone;
this._magView.add_effect(this._inverse);
this._magView.add_effect(this._brightnessContrast);
this._magView.add_effect(this._colorSaturation);
this._magView.add_effect(this._colorDesaturation);
},
/**
@ -1659,7 +1659,7 @@ const MagShaderEffects = new Lang.Class({
*/
destroyEffects: function() {
this._magView.clear_effects();
this._colorSaturation = null;
this._colorDesaturation = null;
this._brightnessContrast = null;
this._inverse = null;
this._magView = null;
@ -1684,11 +1684,11 @@ const MagShaderEffects = new Lang.Class({
},
setColorSaturation: function(factor) {
this._colorSaturation.set_factor(factor);
this._colorDesaturation.set_factor(1.0 - factor);
},
getColorSaturation: function() {
return this._colorSaturation.get_factor();
return 1.0 - this._colorDesaturation.get_factor();
},
/**