New design doc: Architecture of material_ui and cupertino_ui#13529
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a new redirect entry in firebase.json for /go/material_ui-cupertino_ui-architecture. The feedback suggests replacing underscores with hyphens in the source path to maintain alphabetical sorting and consistency with existing URL patterns.
|
Staged preview of the updated docs.flutter.dev site (updated for commit cab72f5): https://flutter-docs-prod--docs-pr13529-dkwingsmt-patch-1-qy6w350s.web.app |
|
Staged preview of the updated flutter.dev site (updated for commit cab72f5): https://flutter-dev-230821--www-pr13529-dkwingsmt-patch-1-x32f3flh.web.app |
parlough
left a comment
parlough
left a comment
There was a problem hiding this comment.
Do consider using dashes in the redirect. Otherwise looks good to me. Thanks!
|
I don't hold strong opinions but I chose underscores because |
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Yes, as that's generally how we format URL paths, even for package names. For consistency, I went ahead and pushed the update to Thanks! |
|
Sounds good. Thank you! |
3 participants
Add
/go/material-ui-cupertino-ui-architecturedesign doc redirect.