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
Adding the basis for the Logging v2 experience #7724
base: master
Are you sure you want to change the base?
Conversation
packages/devtools_app/lib/src/screens/logging/logging_screen_v2/logging_screen_v2.dart
Show resolved
Hide resolved
packages/devtools_app/lib/src/screens/logging/logging_screen_v2/logging_screen_v2.dart
Show resolved
Hide resolved
packages/devtools_app/test/logging/logging_screen_v2/logging_controller_v2_test.dart
Show resolved
Hide resolved
packages/devtools_app/test/logging/logging_screen_v2/logging_screen_v2_test.dart
Show resolved
Hide resolved
I made another version that just copies the controller and screen over, so this PR will be more simple. |
packages/devtools_app/lib/src/screens/logging/logging_screen_v2/logging_controller_v2.dart
Outdated
Show resolved
Hide resolved
packages/devtools_app/lib/src/screens/logging/logging_screen_v2/logging_controller_v2.dart
Outdated
Show resolved
Hide resolved
Row( | ||
children: [ | ||
Expanded( | ||
child: TextField( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use DevToolsClearableTextField for both of these. Fine to implement functionality later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll implement this in the next one.
with | ||
AutoDisposeMixin, | ||
ProvidedControllerMixin<LoggingControllerV2, LoggingScreenBody> { | ||
ProvidedControllerMixin<LoggingControllerV2, LoggingScreenBodyV2> { | ||
List<String> items = []; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: logs instead of items
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be part of upcoming changes.
Copied and modified the logging screen and logging controller files.
This changes is just meant to establish the main points so we can build each of the components in the next PRs.