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

iOS 18 compatible OBA Widget Extension #753

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

manu-r12
Copy link

Description:

  • Added a WidgetKit widget to display transit arrivals and departures at a glance.
  • Optimized for iOS 18 compatibility.
  • The widget refreshes every 30 minutes or can be refreshed through user interaction.

Media:

Preview -

Screenshot 2024-10-23 at 3 19 37 PM

@manu-r12
Copy link
Author

@aaronbrethorst - can you please review it.

@manu-r12
Copy link
Author

Hey @aaronbrethorst, if you need any help, just let me know. I'd be happy to volunteer!.

@aaronbrethorst
Copy link
Member

@manu-r12 sorry I didn’t have a chance to review today! It’s at the top of my todo list for tomorrow. Unexpectedly lost childcare all this week so finding time for … well, everything has been a challenge.

@manu-r12
Copy link
Author

Hey @aaronbrethorst , no worries at all, life happens, and I totally understand. Please take your time ❤️

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

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

This is looking really good! I have noted a couple questions and some minor changes. Once you have those sorted out, I'll do a final UI review and we'll get this merged!

OBAWidget/Components /RefreshButton.swift Outdated Show resolved Hide resolved
OBAWidget/Components /RefreshButton.swift Outdated Show resolved Hide resolved
OBAWidget/Entries/BookmarkEntry.swift Outdated Show resolved Hide resolved
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.

2 participants