Skip to content

Conversation

@parlough
Copy link
Member

@parlough parlough commented Dec 19, 2024

  • Fix a few minor typo, punctuation, and grammar issues
  • Adjust wording for clarity
  • Add more cross links to Flutter, Firebase, and package docs where relevant
  • Add alt text to screenshot and make it a link target as well
  • Adjust sections of text that are links
  • Use Markdown and GFM features where appropriate (ordered list, warning blocks, etc)
  • Add some line breaks and separate link definitions for easier editing and review
  • Update comments in code blocks to more closely follow Effective Dart
  • Adjust placeholders (...) in code snippets to be in comments to make them directly copyable
  • Other misc and minor adjustments

@csells
Copy link
Contributor

csells commented Dec 19, 2024

When I preview the updated README in this PR, the paragraph flow is gone and all of the lines are unnaturally shortened:

image

@domesticmouse
Copy link
Contributor

Please resolve conflicts =)

@parlough
Copy link
Member Author

parlough commented Jan 6, 2025

When I preview the updated README in this PR, the paragraph flow is gone and all of the lines are unnaturally shortened:

This is intentional to use semantic breaks for making editing and reviewing raw Markdown easier and more intentional.

Markdown renderers join lines with spaces so these awkward previews won't show up in most cases, but it seems the VS Code preview you're using doesn't handle that properly. You can see in the GitHub view those line breaks don't appear.

@domesticmouse domesticmouse merged commit 521fe50 into main Jan 6, 2025
3 checks passed
@domesticmouse domesticmouse deleted the feat/readme-cleanup branch January 6, 2025 22:45
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.

4 participants