focus fade with light palette color
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Sakura |
Fix Released
|
Medium
|
David Gómez |
Bug Description
Hi,
When using a light palette like "Solarized light" with a light background and a dark foreground then the focus fade feature tries to darker the foreground which is already dark.
So it is hard to differentiate fade terminals and not fade terminals.
In this case, I tried to change the source code and change the background color instead of the foreground and it's much better; I did the following hack that only works in my case (it does not work with dark palette colors):
#######
--- a/src/sakura.c
+++ b/src/sakura.c
@@ -308,7 +308,7 @@ struct terminal {
#define TAB_MIN_SIZE 6
#define FORWARD 1
#define BACKWARDS 2
-#define FADE_PERCENT 60
+#define FADE_PERCENT 90
#define DEFAULT_
#define DEFAULT_
#define DEFAULT_
@@ -2838,12 +2838,12 @@ sakura_fade_out()
if (!sakura.faded) {
- GdkRGBA x = sakura.
+ GdkRGBA x = sakura.
if ( (x.red >=0 && x.red <=1.0) && (x.green >=0 && x.green <=1.0) && (x.blue >=0 && x.blue <=1.0)) {
- sakura.
+ sakura.
} else {
}
@@ -2862,13 +2862,13 @@ sakura_fade_in()
if (sakura.faded) {
- GdkRGBA x = sakura.
+ GdkRGBA x = sakura.
if ( (x.red >=0 && x.red <=1.0) && (x.green >=0 && x.green <=1.0) && (x.blue >=0 && x.blue <=1.0)) {
- sakura.
+ sakura.
} else {
}
#######
I don't know what can be done to handle all the palette colors (dark and light) easily.
But in the current state, only dark palette colors are working with the fade focus functionality.
Changed in sakura: | |
status: | Fix Committed → Fix Released |
Actually the "Solarized light" support is broken right now. When I fix it (hopefully soon) I'll keep in mind this and make fade work as expected for light palettes.
Thanks for you report!