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

Added app to handle fullscreen notifications #130

Closed
wants to merge 6 commits into from

Conversation

AverageMarcus
Copy link
Contributor

@AverageMarcus AverageMarcus commented Mar 18, 2020

Added an application that can be triggered by other Bangle.js apps to display fullscreen notifications and allows the wearer to look back at previous notifications (configurable).

Screenshots:

image

image

image

@MaBecker
Copy link
Contributor

MaBecker commented Mar 18, 2020

think about adding g.reset() to be save.

@MaBecker Not sure what you mean by "to save". Doesn't g.reset() put all the graphics settings back to the default as on load?

exactly: everything to default

@AverageMarcus
Copy link
Contributor Author

@MaBecker Not sure what you mean by "to save". Doesn't g.reset() put all the graphics settings back to the default as on load?

@MaBecker
Copy link
Contributor

just update my first comment

@gfwilliams
Copy link
Member

This looks like a really neat solution - thanks!

Please could you just tweak the changelog into the text-only format used for the other logs though? The loader looks for a ChangeLog file and loads it when the version is clicked on and I'd rather not have to check for a bunch of different styles :)

@AverageMarcus
Copy link
Contributor Author

Updated. Is this the right format @gfwilliams?

@gfwilliams
Copy link
Member

Yes, that's perfect - thanks!

@Xaseron
Copy link
Contributor

Xaseron commented Mar 30, 2020

Hey, greetings from germany.
Can we change the font to Dennis8 http://www.espruino.com/Fonts?
This would help to display notfication messages with special characters like äöüß

@MaBecker
Copy link
Contributor

Hey, greetings from germany.
Can we change the font to Dennis8 http://www.espruino.com/Fonts?
This would help to display notfication messages with special characters like äöüß

Well, the 6x8 font has all you need.

espruino/Espruino#1770

@Xaseron
Copy link
Contributor

Xaseron commented Mar 31, 2020

Awesome.
I've haven't tested the gadgebridge releated PRs.
I just can state that the current available packages in app loaders have problems with special characters. Maybe the cause is gadgetbridge on my phone. I don't know.

@gfwilliams gfwilliams closed this in dc8d2fc Jun 4, 2020
@gfwilliams
Copy link
Member

Sorry it's taken so long to sort this, it seemed tricky to integrate this nicely.

I'm afraid I haven't merged this, but have added a simple notify (top of screen as before) and notifyfs app, which create something that allows you to require("notify").show(...). Apps can also set a dependency so that a notification library gets loaded automatically if one doesn't exist.

There's no stored notification list but hopefully the new one can run at the same time as most other apps. The issue with this PR (as far as I could see) was once it's fired it doesn't load widgets, so you wouldn't actually get a second notification if one came in?

If you'd like to PR the new notifyfs with any changes that'd be great though!

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.

None yet

4 participants