[Vorschlag] LCD / Standby Helligkeit

[GetAway]

Hi,

viele haben sich mittlerweile die neuen billigen Displays eingebaut.
Diese sind teilweise superhell und strahlen in alle Richtungen.
Der kleinste Wert der Standby-Helligkeit ist ja "5".
Mein Vorschlag wäre nun, ab dem Wert 5 in Einerschritten runter
bis auf "1" zu gehen. Das bringt bei den neuen Displays einiges.

Gruß GetAway

[seife]

Oder einfach generell in Einserschritten einstellbar machen. Ich habe den Code schon fertig (wird auch noch einfacher dadurch ), aber ich wills am Wochenende erst ausprobieren, vor ich's einchecke.

[GetAway]

Wenn der Wertebereich bis 255 bleibt! Sonst bringts ja nichts.

[mb405]

eigentlich brauch mal nur in der lcdcontroler.cpp die +5 -5 und die 250 bzw 5 ersetzen.
aber wie ich seife so kenne, baut er bei der gelegenheit den ganzen code um
ich habs so gemacht. denke das geht so auf die schnelle. alte werte durch /* .. */ ausgeklammert

Code: Alles auswählen

case 1:
						if (brightness < 255)
						{
							if (brightness < /*250*/255)
								brightness += /*5*/1;
							else
								brightness = 255;

							paintSlider(x+10, y+hheight+mheight, brightness, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESS, true);
							setLcd();
						}
						break;
					case 2:
						if (brightnessstandby < 255)
						{
							if (brightnessstandby < /*250*/255)
								brightnessstandby += /*5*/1;
							else
								brightnessstandby = 255;

							paintSlider(x+10, y+hheight+mheight*2, brightnessstandby, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESSSTANDBY, true);
							setLcd();
						}
						break;
....
....
case 1:
						if (brightness > 0)
						{
							if (brightness > /*5*/0)
								brightness -= /*5*/1;
							else
								brightness = 0;

							paintSlider(x+10, y+hheight+mheight, brightness, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESS, true);
							setLcd();
						}
						break;
					case 2:
						if (brightnessstandby > 0)
						{
							if (brightnessstandby > /*5*/0)
								brightnessstandby -= /*5*/1;
							else
								brightnessstandby = 0;

							paintSlider(x+10, y+hheight+mheight*2, brightnessstandby, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESSSTANDBY, true);
							setLcd();
						}
						break;

[mb405]

ich habs grad mal selber eingebaut. also wenn man davon hohen wert runterregeln will is das so doof
am besten ab wert 10 die schrittweite verringern das wäre besser.

[GetAway]

ich habs grad mal selber eingebaut. also wenn man davon hohen wert runterregeln will is das so doof
am besten ab wert 10 die schrittweite verringern das wäre besser.

Das habe ich mir auch gedacht. Mal sehen wie seife es gelöst hat.

[mb405]

naja so schwer is das nich
man muss nur noch ne abfrage rein, wenn contrast < 10 dann mach schrittweite 1 ansonsten 5

[seife]

ich hab's so gemacht:

Code: Alles auswählen

@@ -179,11 +179,7 @@ int CLcdControler::exec(CMenuTarget* par
        case 1:
                if (brightness < 255)
                {
-                       if (brightness < 250)
-                               brightness += 5;
-                       else
-                               brightness = 255;
- 
+                       brightness++;
                        paintSlider(x+10, y+hheight+mheight, brightness, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESS, true);
                        setLcd();
               }
@@ -191,11 +187,7 @@ int CLcdControler::exec(CMenuTarget* par
       case 2:                                                                                                                  
                if (brightnessstandby < 255)                                                                                     
                {                                                                                                                
-                      if (brightnessstandby < 250)                                                                             
-                              brightnessstandby += 5;                                                                          
-                      else                                                                                                     
-                              brightnessstandby = 255;                                                                         
-
+                      brightnessstandby++;                                                                                     
                        paintSlider(x+10, y+hheight+mheight*2, brightnessstandby, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESSSTANDBY, true);
                        setLcd();
               }
@@ -217,11 +209,7 @@ int CLcdControler::exec(CMenuTarget* par
        case 1:
                if (brightness > 0)
                {
-                       if (brightness > 5)
-                               brightness -= 5;
-                       else
-                               brightness = 0;
-
+                       brightness--;
                        paintSlider(x+10, y+hheight+mheight, brightness, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESS, true);
                        setLcd();
                }
@@ -229,11 +217,7 @@ int CLcdControler::exec(CMenuTarget* par
        case 2:
                if (brightnessstandby > 0)
                {
-                       if (brightnessstandby > 5)
-                               brightnessstandby -= 5;
-                       else
-                               brightnessstandby = 0;
-
+                       brightnessstandby--;
                        paintSlider(x+10, y+hheight+mheight*2, brightnessstandby, BRIGHTNESSFACTOR, LOCALE_LCDCONTROLER_BRIGHTNESSSTANDBY, true);
                        setLcd();
                }

Damit fällt die doppelte Abfrage nach >0 und >5 z.B. weg. Aber genau weil ich es noch nicht auf der Hardware ausprobiert habe, habe ich es noch nicht eingecheckt.
Am besten wäre es ja, bei einem kurzen/einzelnen Tastendruck eins hoch und runterzuzählen, und beim key-repeat jeweils 5 oder 10. Aber das geht in neutrino nicht so leicht wie z.B. in Enigma

[mb405]

naja beim kontrast wird in einerschritten, weils da ja nur bis 63 geht.
die anderen werte gehen bi 255, da is das ann mühsam wenn man von 255 auf 10 oder umgekehrt machen will
5er sprünge sind da schon ideal, und man könnte ja nen bereich von 0-20 in 1er schritten gehn. das wäre ne alternative

[Tommy]

mit bisschne Mehraufwand sollte mans so machen wie beim Movieplayer spulen:

key1 -=1
key4 -=5
key7 -=10

key3 +=1
key6 +=5
key9 +=10

*duckundwech*