Thanks for the contribution! This PR introduces visual changes, so before and after screenshots should be included. If possible, it's helpful to crop full-screen screenshots to highlight the changes. A test will also be needed.

Unfortunately this change will "break" existing apps that depend on the current implementation's configuration of the search text field. It would be safer to add textStyle and hintStyle parameters.

Thanks for your feedback. I will upload screenshots as soon as possible. @HansMuller Would you add the parameters to the showSearch function or to the SearchDelegate ?

Actually, maybe a better way to address this issue would be for your app to override SearchDelegate.appBarTheme.

Yes but it isn't really easy to override the textSytles. Moreover I think flutter should use the correct Material design. The AppBar uses the primaryTextTheme and I think the TextField should also. Right now I see no possibility to change hintStyle. I added screenshots to the first comment.

I made some changes to the PR. I reverted the TextFiedl style change and the hint style change. Instead I asigned the default hintStyle from the Theme to the hinstyle. Now you can edit the hintStyle, and the changes are backwards compatible.

Merge this already (or #30388).

Want this

Why is this still not merged?

I'm hoping to get #30388 merged soon if everyone can agree in the discussion over there. The solution is the same as this PR, but it includes a test, so I think it's the closest to being merged. Track the status over there in the meantime, and I'll close this if that one is merged.

@justinmc Dude merge it already.

@justinmc It wonders me why nobody answered me after I made the requested changes in this pr. But the newer pr (#30388) gets discussed. It's kind of weird to me. In general pr's from non-contributor receive fewer answers.

But I hope finally one of this prs gets merged.

Yeah sorry about that, I definitely try to give credit to whoever comes up with a solution first, but it's hard to keep track of all of the PRs and issues. This PR is definitely contributing to the sense of urgency of this problem and the validity of the solution, even if it doesn't get merged.

If you open a PR in the future feel free to tag me directly.

@justinmc thanks for the answer. I understand that it's to keep track, it's also frustrating for us developers. But it's nice to hear that you care about what we think.

PS. I am waiting for an answer on a pr at flutter/plugins (pr).:stuck_out_tongue_winking_eye:

Closing this as #30388 was just merged and built successfully.

The solution that went in is the same as proposed in this PR, so SearchDelegate will now respect the appBarTheme's hintStyle. Make sure you're on the master Flutter channel, or using a version > 1.7.3 once it's released.

Thank you again @The-Redhat for starting this whole discussion. I will try to find someone familiar with quick_actions to take a look at that PR :)

Thank you very much @justinmc !