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

Skip to content

Conversation

@micahmo
Copy link
Member

@micahmo micahmo commented Jan 24, 2024

Pull Request Description

This PR builds upon the awesome work done by @ggichure in #1036 and includes a few enhancements to search.

  1. Flatten and adjust height of the search page's search widget to match the MD3 guidelines and the new settings search widget (at some point we might want to consider integrating a SearchAnchor here).
  2. Clear out setting search query when navigating back from page.

Issue Being Fixed

Issue Number: N/A

Screenshots / Recordings

image

Checklist

  • Did you update CHANGELOG.md?
  • Did you use localized strings where applicable?
  • Did you add semanticLabels where applicable for accessibility?

if (focused) FocusScope.of(context).unfocus();
if (focused) {
FocusScope.of(context).unfocus();
_searchController.text = '';
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't clear the text ? settings_page.dart#L111

Copy link
Member Author

Choose a reason for hiding this comment

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

Not for me. 😞 Here is the behavior I see in develop.

qemu-system-x86_64_8ydykT2MeH.mp4

@micahmo
Copy link
Member Author

micahmo commented Jan 24, 2024

I just pushed another commit which fixes the title of the "Scrape Missing Link Previews" setting.

Copy link
Member

@hjiangsu hjiangsu left a comment

Choose a reason for hiding this comment

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

LGTM!

@hjiangsu hjiangsu merged commit 6b0150a into thunder-app:develop Jan 25, 2024
@micahmo micahmo deleted the feature/improve-search branch January 25, 2024 15:13
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