-
Notifications
You must be signed in to change notification settings - Fork 38
Open
Conversation
@AbhishekDoshi26
AbhishekDoshi26
changed the title
(削除) Feat/flutter concepts (削除ここまで)
(追記) feat/state-management (追記ここまで)
Jun 12, 2026
tomarra
tomarra
reviewed
Jun 17, 2026
tomarra
tomarra
reviewed
Jun 17, 2026
tomarra
tomarra
reviewed
Jun 17, 2026
tomarra
tomarra
reviewed
Jun 17, 2026
tomarra
commented
Jun 17, 2026
Contributor
This reads a bit more as a medium article then a doc page. Need to rework the tone a bit here. I left some comments on that but take another pass at it and then I can re-review.
...weather app with the Single State pattern
...attern over multiple state classes
@AbhishekDoshi26
AbhishekDoshi26
force-pushed
the
feat/flutter_concepts
branch
from
June 18, 2026 08:34
a05bd55 to
430e6e4
Compare
tomarra
tomarra
approved these changes
Jun 18, 2026
@tomarra
tomarra
left a comment
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good with this for now. Same comment applies from other reviews, feels like there is a lot of code in here that probably needs to live in an example repo/app to point to which also forces us to keep it up to date with latest version of Flutter/Dart instead of static code that gets stale in docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR restructures the State Management section under the Flutter Concepts category. It splits the content into two distinct guides (Beginner and Advanced) and refactors both guides to standardize on the Single State + Enum Pattern as the recommended industry best practice.
Changes Made
📁 Reorganization & Sidebar Styling
src/content/docs/flutter-concepts/state-management/directory and added a_meta.ymlconfiguration to define the sidebar group.src/styles/custom.cssto make nested sidebar group headers normal font-weight and match the style/color of regular article links.state-management-with-flutter.mdx.📄 Beginner Guide: State Management in Flutter
beginner.mdxbased on the Medium article on BLoC Single State concepts.📄 Advanced Guide: Advanced State Management (BLoC)
advanced.mdxby adapting the weather app BLoC tutorial.WeatherStatusenum and a singleWeatherStateclass.WeatherBlocevent handlers to emit copied states.WeatherViewUI widget wiring to retain weather data in memory during background refreshes.Verification