aboutsummaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Systemtask : replace enum SystemTaskState by an enum class.Jean-François Milants2022-05-082-25/+25
|
* Simplify systemtask statesRiku Isokoski2022-05-082-37/+34
|
* Disable clip_corner, because it is slow.Riku Isokoski2022-05-081-1/+2
|
* Fix dropdown graphical issues.Riku Isokoski2022-05-082-11/+2
| | | | | | The scrollbar would go out of bounds if DROPDOWN_PART_LIST had uneven padding. Also enable clip_corner feature to stop the selected item from overflowing.
* Force full screen refresh with a rounder functionRiku Isokoski2022-05-0820-92/+22
| | | | | | When the screen switches, the full screen needs to be refreshed for the hardware scrolling to work. This was enforced with backgroundLabels, but is simpler to do with a rounder function.
* Remove lcd from DisplayAppRiku Isokoski2022-05-082-5/+3
|
* List: add missing array includeReinhold Gschweicher2022-05-041-1/+1
| | | | | | | List.h uses `std::array` as container, but is missing the `<array>` include. Add it to make the header self contained. The `memory` include is unused and can be removed.
* Fix large blacklevel step. Lower 25% of shades are now accessible.Riku Isokoski2022-04-299-14/+16
| | | | | | | | | There is a large step in brightness from level zero to level one. After experimenting with various ST7789 options, I found that decreasing VDV to 0x10 (-0.4V) fixes this issue. The gamma change reduced the average error in brightness, but with the underlying issue fixed, the gamma change has been reverted.
* New sharper batteryicon. Remove old unused batteryiconsRiku Isokoski2022-04-2561-1392/+145
|
* Fix warningsRiku Isokoski2022-04-2511-25/+25
|
* Replace app_timer with FreeRTOS timersRiku Isokoski2022-04-253-64/+29
|
* Keep the paddle on screen in wholeReinhold Gschweicher2022-04-251-1/+8
| | | | | | | | As suggested in https://github.com/InfiniTimeOrg/InfiniTime/issues/1062 in point 3. The paddle should be kept on screen and not travel out of the screen boundaries. Co-authored-by: Riku Isokoski <riksu9000@gmail.com>
* Make functions const againRiku Isokoski2022-04-252-2/+2
|
* Remove now unused position overrideRiku Isokoski2022-04-251-3/+0
| | | | It gets overridden later anyway
* Update track progress in MusicService. Fix #127Riku Isokoski2022-04-254-28/+27
|
* Improve PTS color matching after gamma changeRiku Isokoski2022-04-251-6/+6
|
* Tweak gamma on ST7789 and match UI colorsRiku Isokoski2022-04-2529-88/+96
| | | | | This change will increase the color accuracy of the PineTime and make UI development with the simulator easier.
* Use centisecondsRiku Isokoski2022-04-251-4/+5
|
* Stopwatch fixesRiku Isokoski2022-04-252-37/+22
|
* Fix timer layoutRiku Isokoski2022-04-251-16/+11
|
* Firmware update screen fixesRiku Isokoski2022-04-251-11/+10
|
* SpiNorFlash: use C++ style struct in C++ only headerReinhold Gschweicher2022-04-241-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | `SpiNorFlash.h` is a C++ header, but the `Identification` struct is created in a C style using `typedef struct`. Clang issues a warining about this discrepancy: ``` In file included from /home/nero/repos/pinetime/InfiniSim/InfiniTime/src/systemtask/SystemTask.cpp:13: /home/nero/repos/pinetime/InfiniSim/sim/drivers/SpiNorFlash.h:16:21: warning: anonymous non-C-compatible type given name for linkage purposes by typedef declaration; add a tag name here [-Wnon-c-typedef-for-linkage] typedef struct __attribute__((packed)) { ^ Identification /home/nero/repos/pinetime/InfiniSim/sim/drivers/SpiNorFlash.h:17:9: note: type is not C-compatible due to this default member initializer uint8_t manufacturer = 0; ^~~~~~~~~~~~~~~~~~~~ /home/nero/repos/pinetime/InfiniSim/sim/drivers/SpiNorFlash.h:20:9: note: type is given name 'Identification' for linkage purposes by this typedef declaration } Identification; ^ 1 warning generated. ``` The easy fix is to use a C++ style struct. Also includes code style fix from Riksu9000 Co-authored-by: Riku Isokoski <riksu9000@gmail.com>
* Merge branch 'develop' into remove-nm-referenceMax Friedrich2022-04-1990-1072/+1584
|\
| * Replace lv_label_set_text where possibleRiku Isokoski2022-04-1822-104/+98
| |
| * Merge pull request #1009 from ↵Riku Isokoski2022-04-181-1/+2
| |\ | | | | | | | | | | | | NeroBurner/AlarmController_allow_less_precice_system_time AlarmController: allow loss of precision for alarmTime cast
| | * AlarmController: allow loss of precision for alarmTime castReinhold Gschweicher2022-03-291-1/+2
| | | | | | | | | | | | | | | Allow a loss of precision if the system clock has a lower resolution than nanoseconds. This is the case for web assembly.
| * | resolved merge conflict after renaming PineTimeStyle to WatchFacePineTimeStylemabuch2022-04-1829-175/+182
| |\ \
| | * | Let TouchHandler return TouchEvents instead of driver specific enumReinhold Gschweicher2022-04-134-33/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let the TouchHandler::GestureGet() function return a TouchEvent instead of the touchpanel-driver specific enum. This helps to move the driver specific helper function `ConvertGesture` from `DisplayApp` into `TouchHandler`.
| | * | Replace airplane mode with a bluetooth toggleRiku Isokoski2022-04-0214-90/+72
| | |/
| | * Timer App : add background label to ensure that the app will be displayed ↵Jean-François Milants2022-03-283-20/+28
| | | | | | | | | | | | | | | | | | correctly after a full refresh (HW scrolling transition). Code cleaning and rename methods.
| | * DisplayApp : Call the event handler of the current app before loading the ↵Jean-François Milants2022-03-281-4/+4
| | | | | | | | | | | | new one. This way, we ensure that lv_task_handler() is called before sending event to the newly loaded app.
| | * New changes according to the review : Priority 0 for display, 1 for system, ↵Jean-François Milants2022-03-213-3/+4
| | | | | | | | | | | | timer and ble host, and 2 for ble LL
| | * Fix priorities of BLE tasksJean-François Milants2022-03-211-2/+2
| | |
| | * In current configuration, the timer task (the one from FreeRTOS) has the ↵Jean-François Milants2022-03-212-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | lowest priority (0). Both display and system tasks are also set on priority 0. In cases where any other task takes too much time to execute (it can happen in Display Task, see https://github.com/InfiniTimeOrg/InfiniTime/issues/825), the timer task does not have the opportunity to run fast enough to detect and debounce presses on the button. This commit sets the following priorities: - [0] : Display Task - [1] : Timer and System tasks - [2] : BLE Host - [3] : BLE LL This way, we ensure that button presses will always be detected, even if the rendering of the display takes a huge amount of time.
| | * Music app : when title/track name are truncated, add an ellipsis at the end ↵Jean-François Milants2022-03-211-5/+13
| | | | | | | | | | | | of the strings.
| | * Add missing space in if expression.Jean-François Milants2022-03-211-1/+1
| | |
| | * Limit the size of the track and album name received by MusicService. This ↵Jean-François Milants2022-03-211-0/+7
| | | | | | | | | | | | should work around this bug : https://github.com/InfiniTimeOrg/InfiniTime/issues/825 and prevent heap over-allocation.
| | * Merge pull request #1038 from NeroBurner/analog_local_variablesJF2022-03-142-13/+6
| | |\ | | | | | | | | WatchFaceAnalog: local date/time variables
| | | * WatchFaceAnalog: local date/time variablesReinhold Gschweicher2022-03-142-13/+6
| | | | | | | | | | | | | | | | Use local date and time variables. No need to store them in the object.
| | * | Merge pull request #1033 from NeroBurner/music_override_warningJF2022-03-141-1/+1
| | |\ \ | | | | | | | | | | Music: fix warning about overridden function
| | | * | Music: fix warning about overridden functionReinhold Gschweicher2022-03-131-1/+1
| | | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clang warns on `OnTouchEvent()` function, which is overridden, but is missing the `override` keyword ``` In file included from InfiniTime/src/displayapp/screens/Music.cpp:18: InfiniTime/src/displayapp/screens/Music.h:43:14: warning: 'OnTouchEvent' overrides a member function but is not marked 'override' [-Winconsistent-missing-override] bool OnTouchEvent(TouchEvents event); ^ ```
| | * | Merge pull request #1034 from NeroBurner/twos_comparison_warningJF2022-03-141-1/+1
| | |\ \ | | | | | | | | | | Twos: fix warning about extra paranthesis
| | | * | Twos: fix warning about extra paranthesisReinhold Gschweicher2022-03-131-1/+1
| | | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have a comparison like `if (( a == b ))`, which is a parenthesis too much, which generates the following warning ``` InfiniTime/src/displayapp/screens/Twos.cpp:133:35: warning: equality comparison with extraneous parentheses [-Wparentheses-equality] if ((grid[newRow][newCol].value == grid[oldRow][oldCol].value)) { ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ InfiniTime/src/displayapp/screens/Twos.cpp:133:35: note: remove extraneous parentheses around the comparison to silence this warning if ((grid[newRow][newCol].value == grid[oldRow][oldCol].value)) { ~ ^ ~ ```
| | * | WatchFaceAnalog: const ref to dateTimeControllerReinhold Gschweicher2022-03-141-1/+1
| | | | | | | | | | | | | | | | The clock app only reads from the dateTimeController, never modifies it.
| | * | DateTimeController: make possible functions constReinhold Gschweicher2022-03-142-4/+4
| | |/ | | | | | | | | | | | | MonthShortToString and DayOfWeekShortToString don't change the underlying object. Those are just getters and can be declared `const`.
| * / Rename PineTimeStyle to WatchFacePineTimeStylemabuch2022-03-125-22/+22
| |/
| * Merge pull request #1026 from NeroBurner/fix_settings_chimesJF2022-03-091-1/+1
| |\ | | | | | | Fix SettingChimes cbOption array size
| | * Fix SettingChimes cbOption array sizeReinhold Gschweicher2022-03-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | There are 3 options, but the array-size is set to 2. This leads to memory corruption in the initialization of the SettingChimes screen when assigning the third option object pointer. Found in https://github.com/InfiniTimeOrg/InfiniSim/issues/11
| * | Merge pull request #1005 from aveeryy/notifications-as-textJF2022-03-091-6/+6
| |\ \ | | |/ | |/| Terminal watchface: replace notifications icon with a text entry
| | * Remove unnecessary lineavery2022-03-061-1/+0
| | |