Merge pull request #208 from LedgerHQ/fix/207

[fix][#207] Updating Stax ChoiceList positions

CHANGELOG.md

@@ -5,12 +5,16 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [1.24.0] - 2024-09-24
+## [1.24.0] - 2024-10-03
 
 ### Added
 - conftest: Added a all_eink device option to target Stax and Flex
 - conftest: Added a autouse fixture to skip unsupported devices
 
+### Fixed
+
+- firmware/stax: Choice List positions were no longer correct compared to latest NBGL evolutions
+
 ## [1.23.0] - 2024-07-25
 
 ### Changed

src/ragger/firmware/touch/positions.py

@@ -62,16 +62,15 @@ def __iter__(self):
     },
     "ChoiceList": {
         Firmware.STAX: {
-            # Up to 6 (5?) choice in a list
-            1: Position(200, 190),
-            2: Position(200, 260),
-            3: Position(200, 340),
-            4: Position(200, 410),
-            5: Position(200, 480),
-            6: Position(200, 550)
+            # Up to 5 choices in a list
+            1: Position(200, 140),
+            2: Position(200, 235),
+            3: Position(200, 330),
+            4: Position(200, 425),
+            5: Position(200, 520),
         },
         Firmware.FLEX: {
-            # Up to 5 choice in a list
+            # Up to 5 choices in a list
             1: Position(240, 150),
             2: Position(240, 240),
             3: Position(240, 330),