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

Update parallax-scrolling.md #10144

Closed
wants to merge 3 commits into from
Closed

Conversation

afradahsan
Copy link

#10138

In the Parallax scrolling page, in one of the code, there are some unwanted brackets as mentioned in the docs.
I just fixed that and is creating the request.

@sfshaza2
Copy link
Contributor

/gcbrun

@flutter-website-bot
Copy link
Collaborator

Visit the preview URL for this PR (updated for commit a4eb557):

https://flutter-docs-prod--pr10144-patch-1-v4q01pld.web.app

@afradahsan
Copy link
Author

Some checks were not successful
-test / Check excerpts and run tests (Stable channel, stable, false) (pull_request)
-flutter-website-staging (flutter-docs-prod)

Is there something that i should resolve from my side?

@domesticmouse
Copy link
Contributor

This change requires changing the related sample. As the related sample compiles, maybe the issue is a better excerpt could be found? Or maybe the sample need refactoring?

@sfshaza2
Copy link
Contributor

@afradahsan, until you can fix the broken code excerpt, we can't land this.

@sfshaza2
Copy link
Contributor

Superceded by #10180. Thanks for filing, @afradahsan. Closing.

@sfshaza2 sfshaza2 closed this Feb 21, 2024
sfshaza2 added a commit that referenced this pull request Feb 21, 2024
This extends PR #10144 (which I
will close), by removing several extraneous right curly brackets from
Dart samples. It also ran the refresh-code-excepts tool, which updated 3
code excerpts.

Fixes #10138

Thanks, @afradadahsan, for filing the original PR!
atsansone pushed a commit to atsansone/website that referenced this pull request Apr 5, 2024
This extends PR flutter#10144 (which I
will close), by removing several extraneous right curly brackets from
Dart samples. It also ran the refresh-code-excepts tool, which updated 3
code excerpts.

Fixes flutter#10138

Thanks, @afradadahsan, for filing the original PR!
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.

5 participants