8000
Skip to content

Basic support for iOS, fix missing Rust lib #259

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.

Sign up for GitHub

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

Jump to bottom
Merged
KRTirtho merged 1 commit into from
Oct 14, 2022
Merged

Basic support for iOS, fix missing Rust lib #259

KRTirtho merged 1 commit into from
Oct 14, 2022

Conversation

@karniv00l
Copy link
Contributor
@karniv00l karniv00l commented Oct 8, 2022 •
edited
Loading

Related to: #173

Only tested on simulator right now.

Screenshot 2022-10-13 at 19 44 50

Issues:

  • download doesn't work (at least in simulator)
  • playback bar covers bottom menu
  • super weird scrolling / swiping behavior (up/down) - practically unusable (at least in the simulator), I guess this is the change with the Flutter's ios/macos drag physics

karniv00l
karniv00l commented Oct 8, 2022
View reviewed changes
pubspec.yaml
Comment on lines +55 to +58
metadata_god:
git:
url: https://github.com/KRTirtho/metadata_god.git
ref: 7d195fdde324b382fc12067c56391285807e6233
Copy link
Contributor Author
@karniv00l karniv00l Oct 8, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

This is the last working commit

Copy link
Owner
@KRTirtho KRTirtho Oct 9, 2022

Choose a reason for hiding this comment

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

In this commit I guess MacOS wasn't working. Does the MacOS application work?

Copy link
Contributor Author
@karniv00l karniv00l Oct 13, 2022

Choose a reason for hiding this comment

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

This fixes the library on macOS, after this commit it breaks (it can't find the compiled library anymore)

Copy link
Owner
@KRTirtho KRTirtho Oct 13, 2022

Choose a reason for hiding this comment

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

Ah.. my bad. It's actually the latest commit 😅

@KRTirtho
Copy link
Owner
KRTirtho commented Oct 10, 2022 •
edited
Loading

playback overlay covers bottom menu

It is fixed now. Thanks to the new ShellRoute API of go_router. Now it's a part of the bottomNavigationBar. Both will show everywhere except Login Page and PlayerView

@karniv00l
Copy link
Contributor Author
karniv00l commented Oct 13, 2022

It is fixed now. Thanks to the new ShellRoute API of go_router. Now it's a part of the bottomNavigationBar. Both will show everywhere except Login Page and PlayerView

rebased

@KRTirtho KRTirtho merged commit 83792bd into KRTirtho:master Oct 14, 2022
@KRTirtho
Copy link
Owner
KRTirtho commented Oct 14, 2022

Thanks man. Great job ❤️🚀

@karniv00l karniv00l deleted the ios branch October 14, 2022 07:23
@julien-garrigue julien-garrigue mentioned this pull request Dec 1, 2024
Open
2 tasks
@rajarshikhatua rajarshikhatua mentioned this pull request Feb 26, 2025
Closed
2 tasks
@Hat0ru Hat0ru mentioned this pull request Apr 19, 2025
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KRTirtho KRTirtho KRTirtho left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@karniv00l @KRTirtho
0