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

FIX: Broken links on "new-color-scheme-roles.md" page #11245

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

rydmike
Copy link
Contributor

@rydmike rydmike commented Oct 8, 2024

The following four links to reference pages on the Flutter web site page https://docs.flutter.dev/release/breaking-changes/new-color-scheme-roles are broken:

They are broken due to wrong syntax in the markdown file, used URLs are otherwise correct, but the used extended markdown link syntax is not, causing the links to be broken.

This PR fixes the error so the links work.

Presubmit checklist

  • This PR is marked as draft with an explanation if not meant to land until a future stable release.
  • This PR doesn’t contain automatically generated corrections (Grammarly or similar).
  • 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).
  • This PR uses semantic line breaks of 80 characters or fewer.

The following four links to reference pages on the Flutter web site page https://docs.flutter.dev/release/breaking-changes/new-color-scheme-roles are broken:

- [Support fidelity variant for ColorScheme.fromSeed](flutter/flutter#144649)
- [Introduce tone-based surfaces and accent color add-ons - Part 1](flutter/flutter#142654)
- [Introduce tone-based surfaces and accent color add-ons - Part 2](flutter/flutter#138521)
- [Enhance ColorScheme.fromSeed with a new variant parameter](flutter/flutter#144805)

They are broken due to wrong syntax in the markdown file, used URLs are otherwise correct, but the used extended markdown link syntax is not, causing the links to be broken.

This PR fixes the error so the links work.
…eme-roles

FIX: Broken links on "new-color-scheme-roles.md" page
@rydmike rydmike requested review from sfshaza2, parlough and a team as code owners October 8, 2024 12:11
Copy link
Member

@TahaTesser TahaTesser left a comment

Choose a reason for hiding this comment

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

LGTM!

@parlough
Copy link
Member

parlough commented Oct 8, 2024

/gcbrun

@flutter-website-bot
Copy link
Collaborator

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

https://flutter-docs-prod--pr11245-main-4854d77v.web.app

@sfshaza2
Copy link
Contributor

sfshaza2 commented Oct 8, 2024

Yeah, those brackers in the URL are just wrong. However, those links work for me. Weird.

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.

lgtm

@sfshaza2 sfshaza2 merged commit e5ff60a into flutter:main Oct 8, 2024
9 checks passed
@rydmike
Copy link
Contributor Author

rydmike commented Oct 8, 2024

The links with brackets did not work for me, got 404 pages for all of them. Weird. Now they all work. 😄

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