Skip to content

Conversation

msavela
Copy link

@msavela msavela commented Nov 18, 2021

TODO:

  • Disable "Details" search
  • Add breadcrumb navigation to "Widget Details Tree"
  • Render search match highlights (orange highlight for matched string)

@msavela msavela force-pushed the widget-selection-search branch from 964452c to 5009433 Compare November 18, 2021 13:45
alexlindroos and others added 28 commits November 19, 2021 10:55
…#3489)

* Add a method for finding a path from currently selected row item to the tree root (flutter#3489)

* Add InspectorBreadcrumbNavigator (flutter#3489)

* Set text style explicitly (flutter#3489)

* Notify controller about selected widget on tap (flutter#3489)

* Make currently selected item not clickable (flutter#3489)

* Move breadcrumbs out of scrollable tree container (flutter#3489)

* Tweak padding size and appearance parameters (flutter#3489)

* Scroll to the end of the list so the last breadcrumb is always visible (flutter#3489)

* Simplify constructor (flutter#3489)

* Provide inspector tree controller directly to the tree widget where needed (flutter#3489)

* Provide dummy inspectorTreeController to tests in inspector_tree_test (flutter#3489)

* Ensure inspectorTreeController is provided when InspectorTree is showing a widget detail tree (flutter#3489)

* Test if breadcrumbs widget is displayed  (flutter#3489)

* Remove empty constructor (flutter#3489)

* Use row for showing breadcrumbs (flutter#3489)
Disable Details tree search & Highlight search matches with orange color & breadcrumbs
# Conflicts:
#	packages/devtools_app/lib/src/common_widgets.dart
#	packages/devtools_app/lib/src/theme.dart
#	packages/devtools_app/lib/src/ui/search.dart
# Conflicts:
#	packages/devtools_app/lib/src/common_widgets.dart
#	packages/devtools_app/lib/src/network/network_screen.dart
#	packages/devtools_app/test/goldens/timeline_flame_chart_with_selected_frame.png
@tomgilder tomgilder self-requested a review January 28, 2022 10:10
@tomgilder tomgilder marked this pull request as ready for review January 28, 2022 10:10
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.

3 participants