Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Property Editor] Can filter properties in the Property Editor#9022

Merged
elliette merged 14 commits intoflutter:masterfrom
elliette:issue-8955
Mar 20, 2025
Merged

[Property Editor] Can filter properties in the Property Editor#9022
elliette merged 14 commits intoflutter:masterfrom
elliette:issue-8955

Conversation

@elliette
Copy link
Member

@elliette elliette commented Mar 11, 2025

Work towards #8955, #1948

searching

Note: Filed #9006 for search input mis-alignment

@elliette elliette requested a review from a team as a code owner March 11, 2025 18:31
@elliette elliette requested review from kenzieschmoll and removed request for a team March 11, 2025 18:31
@kenzieschmoll
Copy link
Member

DBC, this is actually not search, but filter. The search function leaves the existing data set and allows users to traverse through for matches. Filter removes all parts of the data set that do not match the filter query. See the logging screen for an example of using the StandaloneFilterField widget: https://github.com/flutter/devtools/blob/master/packages/devtools_app/lib/src/screens/logging/logging_controls.dart/#L53

@elliette elliette changed the title [Property Editor] Can search for properties in the Property Editor [Property Editor] Can filter properties in the Property Editor Mar 19, 2025
return Column(
children: <Widget>[
_FilterControls(controller: widget.controller),
if (widget.editableProperties.isEmpty)
Copy link
Member

Choose a reason for hiding this comment

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

what if editableProperties is empty and the filter query is empty? In this case, this message may not make sense

Copy link
Member Author

Choose a reason for hiding this comment

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

That is actually handled before we get here - if there are no editable properties at all we display a message instead of rendering the _PropertiesList:

@elliette elliette merged commit 55fafd2 into flutter:master Mar 20, 2025
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants