Skip to content

Add app info button#88

Closed
RKBoss6 wants to merge 7 commits into
espruino:masterfrom
RKBoss6:appinfo
Closed

Add app info button#88
RKBoss6 wants to merge 7 commits into
espruino:masterfrom
RKBoss6:appinfo

Conversation

@RKBoss6

@RKBoss6 RKBoss6 commented May 23, 2026

Copy link
Copy Markdown
Contributor

This PR adds:

  • a button in each app tile for app information
  • The modal showing:
    • Installs
    • Favorites
    • Device Compaibility
    • (if available) Author
      The reason behind this PR is to add a dedicated menu for each app's information, rather than showing it through a tooltip that won't support mobile users.

I've kept the tooltip intact for now so as to not change too much for users who are still used to the tooltip hover.

It's up on my app loader

Copilot AI review requested due to automatic review settings May 23, 2026 16:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a per-app “info” action to the app tiles, showing a modal with app usage/support metadata (and author when available), to provide a mobile-friendly alternative to hover tooltips.

Changes:

  • Introduces showAppInfo(...) to display an “App Information” modal per app.
  • Refactors tooltip/app metadata construction into a shared getAppInfo(app, expanded) helper.
  • Adds an app-info control to each app tile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread js/index.js Outdated
Comment thread js/index.js Outdated
Comment thread js/index.js Outdated
Comment thread js/index.js Outdated
Comment thread js/index.js
Comment thread js/index.js
Comment thread js/index.js Outdated
@RKBoss6 RKBoss6 marked this pull request as draft May 23, 2026 16:46
@gfwilliams

Copy link
Copy Markdown
Member

I'm not sure if they see this, so @bobrippling and @thyttan - any thoughts?

Personally I'm not super convinced by this - another button makes the app screen look very crowded - and I'm not sure this really solves a problem anyone was complaining about (although can see you can't get the tooltip when on mobile?)

What about keeping the tooltip (it might have to be moved to an actual HTML element as you've done here), but allowing tapping on the app's icon/title to bring it up rather than having a new icon? That way it's not really breaking/changing anything for existing users, but is allowing access on mobile devices.

@RKBoss6

RKBoss6 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Yes, that sounds good, and was actually what I was thinking of doing originally. At least for iOS (not sure about android) there's no way to display tooltips on mobile without custom css tweaks, so I'm thinking we keep the modal and show it when the title/icon is clicked.

@gfwilliams

Copy link
Copy Markdown
Member

Or... we just slap it all into the top of the 'ChangeLog' popup menu? And maybe a 'report issue' button that uses the GitHub file issue link (https://github.com/espruino/BangleApps/issues/new?template=bangle-bug-report-custom-form.yaml&fwversion=...&apps=...) with the author info we now have

Maybe it's overkill having two separate menus for what's effectively just 'app info'

@RKBoss6

RKBoss6 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, I actually like that idea a lot - so instead of showing the changelog history, we show an "app info" modal that has all the information and the changelog. If we want to clean things up even more, we could put the github link in the modal as well instead of in the app tiles, but I'll work on these changes now!

@RKBoss6

RKBoss6 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor Author

Closing this, the new PR is #89

@RKBoss6 RKBoss6 closed this Jun 2, 2026
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.

3 participants