flutter / devtools

Performance tools for Flutter
https://flutter.dev/docs/development/tools/devtools/
BSD 3-Clause "New" or "Revised" License
1.58k stars 326 forks source link

Filters dialog blocks the logs #8485

Open klukshu opened 5 days ago

klukshu commented 5 days ago

I am not sure how much of this is under the purview of VScode vs the flutter DevTools specifically. I'm a software development beginner, but based on the fact that I can't find any information on a better way to do this stuff; at the very least my naivete can inform where documentation could be improved.

DevTools Settings

The log

Filtering the log and a bug

A potential fix

DevTools version: 2.37.2 IDE: VSCode Connected Device: CPU / OS: x64 (64 bit) android Connected app type: Flutter native (debug build) Dart Version: 3.5.1 Flutter Version: 3.24.1 / stable Framework / Engine: 5874a72aa4 / c9b9d5780d

elliette commented 1 day ago

Linking to some duplicate issues for copy-paste here:

Copy-paste in DevTools: https://github.com/flutter/devtools/issues/6794 Copy-paste in DevTools in VS Code IFRAME: https://github.com/microsoft/vscode/issues/129178

I filed https://github.com/flutter/devtools/issues/8496 for the "Enable analytics" concern.

I'll leave this issue open to track addressing the logging concerns:

When the filters dialog is open it blocks the log, so you can't compare spelling to what you are trying to filter out. A potential fix

If there was a right click menu to select whether you want to exclude, or show only that kind of log; it would make it far less tedious to filter out stuff. Ideally you would be able to specify either filter exact matches or logs from the same class. an alternative could be auto fill in the filter menu, or a list of all of the kinds in the log the logical extreme would be to take advantage of the fact that log entries are JSON, they naturally lend themselves to a document oriented database and being queried as such. if this is already the case behind the scenes, the docs need to say what query language is in use.