Skip to content

Conversation

antfitch
Copy link
Contributor

@antfitch antfitch commented Aug 20, 2025

Description of what this PR is changing or adding, and why:

Adds a Rules section and an Overview section.

Previews:

Issues fixed by this PR (if any):

PRs or commits this PR depends on (if any):

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

@antfitch antfitch requested review from sfshaza2, parlough and a team as code owners August 20, 2025 20:02
@antfitch antfitch requested a review from redbrogdon August 20, 2025 20:02
@flutter-website-bot
Copy link
Collaborator

flutter-website-bot commented Aug 20, 2025

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

https://flutter-docs-prod--pr12336-update-create-with-ai-5mzkiqdc.web.app

@antfitch antfitch marked this pull request as draft August 20, 2025 20:18
@antfitch antfitch self-assigned this Aug 20, 2025
@twerske
Copy link

twerske commented Aug 20, 2025

One other q, the double nested header of "Create with AI" is a little funky here, should it be Introduction or something?

image

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far, this is looking great, @antfitch!

@antfitch
Copy link
Contributor Author

One other q, the double nested header of "Create with AI" is a little funky here, should it be Introduction or something?

image

I agree. It does look odd. I don't want to call this an Intro because it's essentially a guide to using AI. I'll look into changing the category title or finding some other title for this topic.

@antfitch
Copy link
Contributor Author

@sfshaza2, @twerske, and @parlough I've made some updates. Please look closely. Here's a quick overview:

  1. Updated category name from "Create with AI" to "AI solutions". This preserves the topic name "Create with AI".
  2. I've used "Rules for Flutter and Dart" in various places, including a section heading.
  3. I've removed "Convert rules between editors" and replaced this with "Create rules for your editor". This is a new space for most people, so I want to help them understand quickly how they can take our template and convert it so that it is useful for various environments.
  4. We don't have the ability to download files from our site (txt, md) that I can see, so @parlough, we'll need to get that figure out with our system.
  5. @twerske we need someone on the development side of things to look very closely at that TXT file I've added. It's essentially the Firebase Studio rules file for Flutter with Firebase Studio bits pulled out.

And that's the summary!

@antfitch antfitch requested a review from twerske August 25, 2025 19:05
@twerske
Copy link

twerske commented Aug 25, 2025

@redbrogdon plans to eval to support item #5 on the list.

Additionally, what are thoughts about putting the download link higher as well of like "Here are the Flutter AI Rules" so it's not only under the subheader section

@antfitch
Copy link
Contributor Author

Additionally, what are thoughts about putting the download link higher as well of like "Here are the Flutter AI Rules" so it's not only under the subheader section

I like this idea. I've done something a little edgy. Maybe it works, maybe it doesn't. Take a close look.

@antfitch antfitch removed the request for review from redbrogdon August 28, 2025 18:19
@antfitch antfitch requested a review from johnpryan August 28, 2025 18:19
@antfitch antfitch marked this pull request as ready for review August 29, 2025 22:12
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