qmk/qmk_firmware

[Bug] Oled scrolling won't resume after oled_timeout.

Open

#20.458 aperta il 15 apr 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C (43.867 fork)batch import
bughelp wanted

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

Describe the Bug

A scrolling oled screen enabled with oled_scroll_left()/oled_scroll_right() won't resume scrolling after OLED_TIMEOUT.

Keyboard Used

draculad

Link to product page (if applicable)

No response

Operating System

No response

qmk doctor Output

No response

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

I took a look into drivers/oled/ssd1306_sh1106.c and the problem seems to be oled_off() does stop the oleds but keeps the oled_scrolling variable to true and due to that, posterior calls to oled_scroll_right() and oled_scroll_left() do nothing.

I fixed it in a naive way ignoring the oled_scrolling check: https://pastebin.com/raw/Z91cjUTma

Guida contributor

Direzione di ricerca
Ispeziona oled off() e oled scroll left/right() in drivers/oled/ssd1306 sh1106.c. La funzione oled off() imposta oled scrolling su true ma non ferma lo scorrimento, quindi le successive chiamate di scorrimento vedono oled scrolling true e vengono saltate. Esamina la correzione fornita su https://pastebin.com/raw/Z91cjUTma: resetta oled scrolling a false in oled off() o rimuovi il controllo nelle funzioni di scorrimento.
Tech stack
c
Dominio
Nessuno
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85