conf.d/10-autohint.conf | 8 +++++++- conf.d/10-no-sub-pixel.conf | 8 +++++++- conf.d/10-sub-pixel-bgr.conf | 8 +++++++- conf.d/10-sub-pixel-rgb.conf | 8 +++++++- conf.d/10-sub-pixel-vbgr.conf | 8 +++++++- conf.d/10-sub-pixel-vrgb.conf | 8 +++++++- conf.d/10-unhinted.conf | 8 +++++++- conf.d/11-lcdfilter-default.conf | 8 +++++++- conf.d/11-lcdfilter-legacy.conf | 8 +++++++- conf.d/11-lcdfilter-light.conf | 8 +++++++- 10 files changed, 70 insertions(+), 10 deletions(-) New commits: commit 1aaf8b773d73f89f5b0d8591cca0a2072524fdb3 Author: Akira TAGOH <akira at tagoh.org> Date: Mon Mar 26 10:58:18 2012 +0900 Bug 17722 - Don''t overwrite user''s configurations in default config Use "append" to avoid overwriting the user configuration. This presumes most clients may takes care of the first value only. diff --git a/conf.d/10-autohint.conf b/conf.d/10-autohint.conf index c597bf4..bf08de8 100644 --- a/conf.d/10-autohint.conf +++ b/conf.d/10-autohint.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Use the Autohinter --> <match target="font"> - <edit name="autohint" mode="assign"><bool>true</bool></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="autohint" mode="append"><bool>true</bool></edit> </match> </fontconfig> diff --git a/conf.d/10-no-sub-pixel.conf b/conf.d/10-no-sub-pixel.conf index 87ada4e..e650160 100644 --- a/conf.d/10-no-sub-pixel.conf +++ b/conf.d/10-no-sub-pixel.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Enable sub-pixel rendering --> <match target="font"> - <edit name="rgba" mode="assign"><const>none</const></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="rgba" mode="append"><const>none</const></edit> </match> </fontconfig> diff --git a/conf.d/10-sub-pixel-bgr.conf b/conf.d/10-sub-pixel-bgr.conf index e1f64fd..74574ad 100644 --- a/conf.d/10-sub-pixel-bgr.conf +++ b/conf.d/10-sub-pixel-bgr.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Enable sub-pixel rendering --> <match target="font"> - <edit name="rgba" mode="assign"><const>bgr</const></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="rgba" mode="append"><const>bgr</const></edit> </match> </fontconfig> diff --git a/conf.d/10-sub-pixel-rgb.conf b/conf.d/10-sub-pixel-rgb.conf index 2b49fe3..3c558e5 100644 --- a/conf.d/10-sub-pixel-rgb.conf +++ b/conf.d/10-sub-pixel-rgb.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Enable sub-pixel rendering --> <match target="font"> - <edit name="rgba" mode="assign"><const>rgb</const></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="rgba" mode="append"><const>rgb</const></edit> </match> </fontconfig> diff --git a/conf.d/10-sub-pixel-vbgr.conf b/conf.d/10-sub-pixel-vbgr.conf index 5947650..392634d 100644 --- a/conf.d/10-sub-pixel-vbgr.conf +++ b/conf.d/10-sub-pixel-vbgr.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Enable sub-pixel rendering --> <match target="font"> - <edit name="rgba" mode="assign"><const>vbgr</const></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="rgba" mode="append"><const>vbgr</const></edit> </match> </fontconfig> diff --git a/conf.d/10-sub-pixel-vrgb.conf b/conf.d/10-sub-pixel-vrgb.conf index 84481bd..1b7b5f6 100644 --- a/conf.d/10-sub-pixel-vrgb.conf +++ b/conf.d/10-sub-pixel-vrgb.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Enable sub-pixel rendering --> <match target="font"> - <edit name="rgba" mode="assign"><const>vrgb</const></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="rgba" mode="append"><const>vrgb</const></edit> </match> </fontconfig> diff --git a/conf.d/10-unhinted.conf b/conf.d/10-unhinted.conf index 025ae2a..67037b4 100644 --- a/conf.d/10-unhinted.conf +++ b/conf.d/10-unhinted.conf @@ -3,6 +3,12 @@ <fontconfig> <!-- Disable hinting --> <match target="font"> - <edit name="hinting" mode="assign"><bool>false</bool></edit> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit name="hinting" mode="append"><bool>false</bool></edit> </match> </fontconfig> diff --git a/conf.d/11-lcdfilter-default.conf b/conf.d/11-lcdfilter-default.conf index 040a4ac..e251edd 100644 --- a/conf.d/11-lcdfilter-default.conf +++ b/conf.d/11-lcdfilter-default.conf @@ -3,7 +3,13 @@ <fontconfig> <!-- Use lcddefault as default for LCD filter --> <match target="font"> - <edit mode="assign" name="lcdfilter"> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit mode="append" name="lcdfilter"> <const>lcddefault</const> </edit> </match> diff --git a/conf.d/11-lcdfilter-legacy.conf b/conf.d/11-lcdfilter-legacy.conf index 218585c..63a4116 100644 --- a/conf.d/11-lcdfilter-legacy.conf +++ b/conf.d/11-lcdfilter-legacy.conf @@ -3,7 +3,13 @@ <fontconfig> <!-- Use lcdlegacy as default for LCD filter --> <match target="font"> - <edit mode="assign" name="lcdfilter"> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit mode="append" name="lcdfilter"> <const>lcdlegacy</const> </edit> </match> diff --git a/conf.d/11-lcdfilter-light.conf b/conf.d/11-lcdfilter-light.conf index d38a66e..af50b2c 100644 --- a/conf.d/11-lcdfilter-light.conf +++ b/conf.d/11-lcdfilter-light.conf @@ -3,7 +3,13 @@ <fontconfig> <!-- Use lcdlight as default for LCD filter --> <match target="font"> - <edit mode="assign" name="lcdfilter"> + <!-- + This configuration is available on the major desktop environments. + We shouldn''t overwrite it with "assign" unconditionally. + Most clients may picks up the first value only. so using "append" + may simply works to avoid it. + --> + <edit mode="append" name="lcdfilter"> <const>lcdlight</const> </edit> </match>