Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[POC] Feature Favorite Apps on swipe left #487

Closed
wants to merge 252 commits into from
Closed

[POC] Feature Favorite Apps on swipe left #487

wants to merge 252 commits into from

Conversation

coxtor
Copy link
Contributor

@coxtor coxtor commented Jul 13, 2021

This is more a POC instead of pretty code, but this PR adds a favorite option when swiping left, including the configurable options.
Issues with this currently:

favorites_demo.mp4

]

Raupinger and others added 30 commits May 16, 2021 00:42
This reverts commit 378fa6b.
Improvement to the mapping of battery percentage to the battery icon to be displayed.
mruss77 and others added 22 commits September 13, 2021 16:05
Remove the unused include `app_timer.h` in `MotorController.h`
…_include

Remove unused include in MotoController.h
Run CI on master and develop branch
`xTaskGetTickCount()` returns a `TickType_t`, which is defined as an
`uint32_t`. This is compared to the `bpm` variable, which is a `int16_t`
in the range of 40 to 220 as defined in the constructor.

```cpp
  lv_arc_set_range(bpmArc, 40, 220);
```

Just assume that `bpm` is greater than 0, as this
would result in a divison by zero or negative values, which would
unintentionally underflow to a very large number.
This reverts commit 1d43adc.
…ime into geekbozu-PersistantStorage

# Conflicts:
#	src/systemtask/SystemTask.cpp
…ng_metronome

Fix unsigned/signed comparison warning in Metronome.cpp
…stored from the noinit area, the date was displayed as "--" instead of the actual date.

This issue was caused by DateTime::SetCurrentTime() that would not update the internal state of the class : dayOfWeek, Month, Year were not properly updated according to the current time.
Toggle notifications only, keep vibrations.
@coxtor
Copy link
Contributor Author

coxtor commented Sep 28, 2021

Hi all, I'm not quite sure what went wrong that this PR has so many commits - however I got to implement the "long tap to select favorite" option. However I need help here - for some reason the watch crashes when I don't use a static Application when saving a favorite (tile.cpp -> apps[buttonId] ) . Can someone help me out here?

@kieranc
Copy link
Contributor

kieranc commented Sep 30, 2021

Any chance you can clean up the PR so we can have a look more easily? I tested this yesterday and it seems to work well but it's a bit of a pain to merge currently

@coxtor coxtor closed this Sep 30, 2021
@coxtor coxtor deleted the favorites branch September 30, 2021 19:22
@coxtor coxtor restored the favorites branch September 30, 2021 19:22
@kieranc
Copy link
Contributor

kieranc commented Sep 30, 2021

I hope you'll bring this back, I was testing it because people were asking for it, and it works nicely!

@coxtor
Copy link
Contributor Author

coxtor commented Sep 30, 2021

I did some cleaning up and we can continue the discussion on PR #708

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.