Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[web] rename dialog to route to match the framework#54228

Merged
auto-submit[bot] merged 3 commits into
flutter-team-archive:mainfrom
yjbanov:rename-dialog-to-route
Jul 31, 2024
Merged

[web] rename dialog to route to match the framework#54228
auto-submit[bot] merged 3 commits into
flutter-team-archive:mainfrom
yjbanov:rename-dialog-to-route

Conversation

@yjbanov

@yjbanov yjbanov commented Jul 30, 2024

Copy link
Copy Markdown
Contributor

Rename "dialog" to "route" to match the framework. The ARIA role "dialog" is an implementation detail, and it might change if a better implementation comes along. The semantic node itself implements the framework's "route" concept.

@yjbanov yjbanov requested a review from chunhtai July 30, 2024 19:52
@github-actions github-actions Bot added the platform-web Code specifically for the web engine label Jul 30, 2024

@chunhtai chunhtai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@yjbanov yjbanov added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 31, 2024
@auto-submit auto-submit Bot merged commit 795c0d8 into flutter-team-archive:main Jul 31, 2024
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Aug 1, 2024
…52648)

Manual roll requested by jacksongardner@google.com

flutter-team-archive/engine@1633272...32f7888

2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter-team-archive/engine#54259)
2024-07-31 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter-team-archive/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (#54215)" (flutter-team-archive/engine#54261)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter-team-archive/engine#54257)
2024-07-31 chris@bracken.jp [iOS][macOS] Eliminate use of bitcode_strip (flutter-team-archive/engine#54254)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter-team-archive/engine#54253)
2024-07-31 matanlurey@users.noreply.github.com Cleanup the shell test, removing unused code (flutter-team-archive/engine#54238)
2024-07-31 yjbanov@google.com [web] rename dialog to route to match the framework (flutter-team-archive/engine#54228)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter-team-archive/engine#54252)
2024-07-31 bdero@google.com [Impeller] Reland: Implement draw order optimization. (flutter-team-archive/engine#54215)
2024-07-31 matanlurey@users.noreply.github.com Migrate the remaining real pub packages to pub workspaces. (flutter-team-archive/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
TytaniumDev pushed a commit to TytaniumDev/flutter that referenced this pull request Aug 7, 2024
…utter#152648)

Manual roll requested by jacksongardner@google.com

flutter-team-archive/engine@1633272...32f7888

2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter-team-archive/engine#54259)
2024-07-31 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter-team-archive/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (flutter#54215)" (flutter-team-archive/engine#54261)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter-team-archive/engine#54257)
2024-07-31 chris@bracken.jp [iOS][macOS] Eliminate use of bitcode_strip (flutter-team-archive/engine#54254)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter-team-archive/engine#54253)
2024-07-31 matanlurey@users.noreply.github.com Cleanup the shell test, removing unused code (flutter-team-archive/engine#54238)
2024-07-31 yjbanov@google.com [web] rename dialog to route to match the framework (flutter-team-archive/engine#54228)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter-team-archive/engine#54252)
2024-07-31 bdero@google.com [Impeller] Reland: Implement draw order optimization. (flutter-team-archive/engine#54215)
2024-07-31 matanlurey@users.noreply.github.com Migrate the remaining real pub packages to pub workspaces. (flutter-team-archive/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Buchimi pushed a commit to Buchimi/flutter that referenced this pull request Sep 2, 2024
…utter#152648)

Manual roll requested by jacksongardner@google.com

flutter-team-archive/engine@1633272...32f7888

2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from 431b57332241 to b5ad5bf3696d (3 revisions) (flutter-team-archive/engine#54259)
2024-07-31 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from cYjTzxe0MskG7PtkB... to uF76DfQgigt4utdBv... (flutter-team-archive/engine#54258)
2024-07-31 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Impeller] Reland: Implement draw order optimization. (flutter#54215)" (flutter-team-archive/engine#54261)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from cd58e87c06f9 to 431b57332241 (2 revisions) (flutter-team-archive/engine#54257)
2024-07-31 chris@bracken.jp [iOS][macOS] Eliminate use of bitcode_strip (flutter-team-archive/engine#54254)
2024-07-31 skia-flutter-autoroll@skia.org Roll Skia from e9e423457655 to cd58e87c06f9 (5 revisions) (flutter-team-archive/engine#54253)
2024-07-31 matanlurey@users.noreply.github.com Cleanup the shell test, removing unused code (flutter-team-archive/engine#54238)
2024-07-31 yjbanov@google.com [web] rename dialog to route to match the framework (flutter-team-archive/engine#54228)
2024-07-31 skia-flutter-autoroll@skia.org Roll Dart SDK from edace067d950 to 5df6a6e0c037 (1 revision) (flutter-team-archive/engine#54252)
2024-07-31 bdero@google.com [Impeller] Reland: Implement draw order optimization. (flutter-team-archive/engine#54215)
2024-07-31 matanlurey@users.noreply.github.com Migrate the remaining real pub packages to pub workspaces. (flutter-team-archive/engine#54232)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from cYjTzxe0MskG to uF76DfQgigt4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine

Development

Successfully merging this pull request may close these issues.

2 participants