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

RouteActivity: Design Idea #190

Draft
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

ugtthis
Copy link
Contributor

@ugtthis ugtthis commented Feb 12, 2025

Code needs work but this is the general idea. Refer to PR comment here.

  • This would get split up into 2 PRs
  • First PR: RouteInfo with expand button that shows RouteActions
  • Second PR: Timeline and RouteVideoPlayer joining with the UI tweaks

@ugtthis ugtthis force-pushed the design-idea--RouteActivity branch from 606b0c1 to 59b6cf4 Compare February 13, 2025 06:31
@ugtthis ugtthis force-pushed the design-idea--RouteActivity branch 2 times, most recently from f0664f0 to 139fa73 Compare February 13, 2025 07:06
Copy link

github-actions bot commented Feb 13, 2025

deployed preview: https://190.connect-d5y.pages.dev

Welcome to new-connect! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

Mobile

Desktop

@incognitojam
Copy link
Member

I like this a lot! Split this up into as many PRs as you like and we can start reviewing/merging

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