tobozo / esp32-DCF77_Analyzer_Clock

⏰ An ESP32 TFT interpretation of the @deruiter's DCF77-Analyzer-Clock-V2.0
MIT License
24 stars 9 forks source link

Leap, CEST & Weekday indicators not showing #4

Open brettoliver opened 4 years ago

brettoliver commented 4 years ago

Hi not sure if these are work in progress but Leap, CEST, week no. & Weekday indicators not showing correctly.

CET is lit when it should be CEST. LEAP is not lit. The weekday indicator is not there-I can see the days of the week but not the actual day is not hi-lighted. The week indicator shows week 28 but is greyed out.

Thanks again for this great project. I have an idea of putting this into a tiny perspex case along with a UDO Klein Super Filter..........

tobozo commented 4 years ago

thanks for your feedback

all those color inversions happened when the display driver was updated to LovyanGFX and the color palette suddenly started to work as expected (the previous driver from the official TFT_eSPI had incorrect init sequence)

I'm quick-and-dirty-fixing this right now and will push changes soon. Also this made me realize there's a strong need for named styles, so I'll probably implement something like this later on, leaving this as a tech debt until refactoring occurs ^^

struct boxStyle {
  std::uint32_t textColor;
  std::uint32_t fillColor;
  boxStyle( std::uint32_t  t, std::uint32_t  f ) : textColor(t), fillColor(f) { }
};

boxStyle *BoxSelected = new boxStyle( TFT_LIGHTGRAY, TFT_WHITE );
boxStyle *BoxUnSelected = new boxStyle( TFT_DARKGRAY, TFT_LIGHTGRAY );