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

Conversation

@bleroux
Copy link
Contributor

@bleroux bleroux commented Feb 17, 2023

Description

This PR replaces obsolete comments, referencing the non-existing ScreenConfiguration type, in ViewConfiguration and replaces them with copied/pasted comments from windows.dart.

Related Issue

Fixes flutter/flutter#120961

Tests

Documentation only.

@bleroux bleroux force-pushed the fix_doc_obsolete_references_to_screen_configuration branch from 2ac2eed to 855ac0c Compare February 17, 2023 09:43
@bleroux bleroux requested a review from gspencergoog February 17, 2023 12:09
@bleroux
Copy link
Contributor Author

bleroux commented Feb 17, 2023

Hey @gspencergoog! Is reusing comments from FlutterView the right fix?

@bleroux bleroux force-pushed the fix_doc_obsolete_references_to_screen_configuration branch from 855ac0c to 4a76031 Compare February 20, 2023 13:08
@chinmaygarde
Copy link
Member

Ping @gspencergoog @goderbauer

Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@goderbauer goderbauer added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 23, 2023
@auto-submit auto-submit bot merged commit 2d2b51d into flutter:main Feb 23, 2023
@bleroux bleroux deleted the fix_doc_obsolete_references_to_screen_configuration branch February 23, 2023 21:35
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 23, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Feb 23, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Feb 23, 2023
…121365)

* 2bc07cac7 Roll Skia from a321a8750271 to 1b2d815e9778 (5 revisions) (flutter/engine#39821)

* 57cbf0858 Roll Skia from 1b2d815e9778 to fff6c987d803 (2 revisions) (flutter/engine#39823)

* 2d2b51d69 Remove obsolete references in ViewConfiguration documentation (flutter/engine#39708)

* cad5eec1b Match PlatformConfiguration properties to PlatformDispatcher ones (flutter/engine#39685)
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Obsolete references in ViewConfiguration documentation

3 participants