Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
9b25e71
Log everywhere I see onBackPressed... And it doesn't log anything!
justinmc Jan 27, 2023
b10efe3
Just hackily remove all usage of onBack, and predictive back does wor…
justinmc Jan 27, 2023
75d8e38
Undid all of the commenting and deleting and located the one call tha…
justinmc Jan 27, 2023
e8a5130
No logging verbose
justinmc Jan 27, 2023
9c0e7cc
WIP A platform channel API to declare that the nav stack is empty
justinmc Jan 28, 2023
205d1e5
navigationStackIsEmpty method to enable predictive back
justinmc Feb 1, 2023
66d1f84
Migrate to updateNavigationStackStatus, works for inner routes
justinmc Feb 9, 2023
9e8e230
Clean up logs and todos
justinmc Feb 9, 2023
0ed2bec
Rename hasMultiple to frameworkHandlesPop for clarity
justinmc Mar 7, 2023
2afbbad
Merge branch 'main' into predictive-back-root
justinmc Mar 7, 2023
8aeba57
Analyzer fix
justinmc Mar 7, 2023
5e89211
WIP some debugging
justinmc Apr 7, 2023
015f2a5
Merge branch 'main' into predictive-back-root
justinmc Apr 7, 2023
b7b2f17
Merge branch 'main' into predictive-back-root
justinmc May 12, 2023
6bc7d5e
Rename to frameworkHandlesBacks
justinmc May 26, 2023
01a22a2
Clean up
justinmc May 26, 2023
6f22009
Confirmed that it works on my old (API 24) samsung device
justinmc May 26, 2023
440a6ee
I dont think I need to modify flutterfragment(activity)
justinmc May 30, 2023
e075275
Tests and empty implementation for FlutterFragment
justinmc Jun 2, 2023
af98e2c
Merge remote-tracking branch 'upstream/main' into predictive-back-root
justinmc Jun 5, 2023
1db22e5
More specific name for hasRegisteredCallback
justinmc Jun 7, 2023
4701d66
Backs to Back
justinmc Jun 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
navigationStackIsEmpty method to enable predictive back
  • Loading branch information
justinmc committed Feb 8, 2023
commit 205d1e53397591594e400b79e16280d62b84f922
Original file line number Diff line number Diff line change
Expand Up @@ -666,14 +666,21 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
public void registerOnBackInvokedCallback() {
if (Build.VERSION.SDK_INT >= 33) {
// TODO(justinmc): This is really the one and only thing that stops root
// pback for me.
// pback for me. There are probably other places you need to mess with
// this for other embedding strategies!
Log.e("justin", "registerOnBackInvokedCallback");
getOnBackInvokedDispatcher()
.registerOnBackInvokedCallback(
OnBackInvokedDispatcher.PRIORITY_DEFAULT, onBackInvokedCallback);
}
}

@Override
public void navigatorIsEmpty() {
Log.e("justin", "navigatorIsEmpty in embedding FlutterActivity");
unregisterOnBackInvokedCallback();
}

/**
* Unregisters the callback from OnBackInvokedDispatcher.
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1661,6 +1661,13 @@ public boolean popSystemNavigator() {
return false;
}

@Override
public void navigatorIsEmpty() {
// TODO(justinmc): Implement. Implementing in FlutterActivity first.
// unregisterOnBackInvokedCallback();
Log.e("justin", "navigatorIsEmpty in embedding FlutterFragment");
}

@VisibleForTesting
@NonNull
boolean shouldDelayFirstAndroidViewDraw() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,8 @@ public interface PlatformPluginDelegate {
* androidx.activity.OnBackPressedDispatcher} will be executed.
*/
boolean popSystemNavigator();

void navigatorIsEmpty();
}

@VisibleForTesting
Expand Down Expand Up @@ -111,6 +113,7 @@ public void setSystemUiOverlayStyle(

@Override
public void navigatorIsEmpty() {
Log.e("justin", "navigatorIsEmpty in PlatformHandler.PlatformMessageChannel.");
PlatformPlugin.this.navigatorIsEmpty();
}

Expand Down Expand Up @@ -482,8 +485,7 @@ private void setSystemChromeSystemUIOverlayStyle(

private void navigatorIsEmpty() {
Log.e("justin", "navigatorIsEmpty in PlatformPlugin");
// TODO(justinmc): Can't quite call it like this, need FlutterActivity.
// activity.unregisterOnBackInvokedCallback();
platformPluginDelegate.navigatorIsEmpty();
}

private void popSystemNavigator() {
Expand Down