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

Revert "Update layout tutorial for current best practices" #9927

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

parlough
Copy link
Member

@parlough parlough commented Dec 14, 2023

Reverts #9893

Some code blocks were unintentionally changed, image styles on other pages were broken, and at least one of the diff code blocks isn't rendering.

Fixes #9925

\cc @atsansone

@flutter-website-bot
Copy link
Collaborator

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

https://flutter-docs-prod--pr9927-revert-9893-fix-2108-hypjcol8.web.app

@parlough
Copy link
Member Author

parlough commented Dec 14, 2023

\cc @domesticmouse For approval since this is currently resulting in a few user experience issues and it'll be easier to land this revert (and fix in a new PR) than trying to fix them all now. Thank you!

@domesticmouse domesticmouse merged commit 4579089 into main Dec 14, 2023
9 checks passed
@domesticmouse domesticmouse deleted the revert-9893-fix-2108 branch December 14, 2023 03:40
atsansone added a commit that referenced this pull request Dec 15, 2023
atsansone added a commit to atsansone/website that referenced this pull request Dec 15, 2023
atsansone added a commit to atsansone/website that referenced this pull request Dec 19, 2023
atsansone added a commit that referenced this pull request Dec 20, 2023
…tes (#9937)

Reverts #9927

Fixes #2108
Fixes #1799 
Fixes #9118 

Based on discussion with @parlough .

- #9925 caused by adding
`width: 100%` to `site-figure-container`. Likely should be `max-width`.
Need to change one line in this.

```css
  .site-figure-container {
    flex: 0 1 auto;
    width: 100%;
    max-width: 100%;
```  

- The code block in the `Update the app to display the button section`
section failed to render.
- Step 4 of the [Interactivity
page](https://docs.flutter.dev/ui/interactivity#step-4-plug-the-stateful-widget-into-the-widget-tree)
was changed so the diff no longer makes sense.

---------

Co-authored-by: Kenzie Davisson <43759233+kenzieschmoll@users.noreply.github.com>
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.

"Adding iOS app extensions" screenshots are distorted
3 participants