Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Added back button interceptor#182

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
feduke-nukem wants to merge6 commits intonslogx:develop
base:develop
Choose a base branch
Loading
fromfeduke-nukem:prevent-back-button-pop
Open

Added back button interceptor#182

feduke-nukem wants to merge6 commits intonslogx:developfromfeduke-nukem:prevent-back-button-pop

Conversation

@feduke-nukem
Copy link

@feduke-nukemfeduke-nukem commentedAug 13, 2022•
edited
Loading

Closes#178
Closes#114
Closes#86

As I myself have faced this problem, I added BackButtonInterceptor package, so if it's declared no userInteractions, there will be no route popping

hmG3 reacted with rocket emoji
Added BackButtonInterceptor package, so if it's declared no userInteractions, there will be no route popping
Added same logic for showProgressReadme, changelog upd
Reorgonized adding interceptor
@RogerBrusamarello
Copy link

@nslogx please could you aprove this PR?

@feduke-nukemfeduke-nukem closed this by deleting the head repositoryApr 3, 2023
@hellohejinyu
Copy link

image

image

@feduke-nukem Dude, you deleted your fork and my code crashed. Because I'm directly referencing your branch code😭

@feduke-nukem
Copy link
Author

imageimage

@feduke-nukem Dude, you deleted your fork and my code crashed. Because I'm directly referencing your branch code😭

Sorry, I was doing some cleanup and didn't intend to break anyone's code. This PR is almost a year old, so I believe the code owner should take a look at it, at least. From my side, I will try to restore this repo and branch asap.

@hellohejinyu
Copy link

Haha thanks, but I've re-forked one from your codebase. Considering that other developers may have used your code, you can indeed restore it.😊

feduke-nukem added a commit to feduke-nukem/flutter_easyloading that referenced this pull requestApr 4, 2023
@feduke-nukem
Copy link
Author

feduke-nukem commentedApr 4, 2023•
edited
Loading

Haha thanks, but I've re-forked one from your codebase. Considering that other developers may have used your code, you can indeed restore it.😊

You could check the branch now.

Additionally, I would like to mention that I have been working on my own implementation of an overlay-based dialog system in Flutter. You might want to take a looklink

My main goal is to provide a flexible system, so I would be happy to receive feedback from anyone.

hellohejinyu reacted with thumbs up emoji

@hellohejinyu
Copy link

You could check the branch now.

Additionally, I would like to mention that I have been working on my own implementation of an overlay-based dialog system in Flutter. You might want to take a look.:link

My main goal is to provide a flexible system, so I would be happy to receive feedback from anyone.

Wow,flutter_easy_dialogs is really cool, I think I can use it in our app in the future.

feduke-nukem reacted with hooray emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

1 more reviewer

@js2762js2762js2762 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

4 participants

@feduke-nukem@RogerBrusamarello@hellohejinyu@js2762
[8]ページ先頭
©2009-2025 Movatter.jp