Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SearchView #147

Merged
merged 7 commits into from
Apr 20, 2020
Merged

Add SearchView #147

merged 7 commits into from
Apr 20, 2020

Conversation

danirabbit
Copy link
Member

@danirabbit danirabbit commented Apr 14, 2020

Fixes #98
Fixes #139

Screenshot from 2020-04-13 20 37 31@2x

@danirabbit danirabbit added the Needs Design Waiting for input from the UX team label Apr 14, 2020
@danirabbit danirabbit added this to In progress in Settings Search via automation Apr 14, 2020
Copy link
Contributor

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

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

We probably need to ellipsize this, but otherwise it is looking good. It would also be nice to get the bold text like in Applications Menu, but that can be a follow-up. Nice work so far!

image

(Demonstrated with https://github.com/elementary/switchboard-plug-about/tree/long-search-string-demo)

@danirabbit
Copy link
Member Author

@cassidyjames oops. You think middle ellipsize is probably good for that?

@cassidyjames
Copy link
Contributor

@danrabbit yeah probably. We end-ellipsize in Applications Menu but I think middle will help ensure the most precise bits are visible.

Copy link
Contributor

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

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

Another note: because the search entry gets disabled, you can get it into a weird state if you're in a search and then try to open a plug from a settings:// URL, i.e. from an indicator or the Applications Menu search—there's no way to get out of the search to see the open plug without clicking a result or hitting the back button.

image

Instead, we should make sure to close the search when opening a plug.

Edit: oops, ignore my playing with alignment here, I was testing some unrelated things locally 😅

Copy link
Contributor

@cassidyjames cassidyjames left a comment

Choose a reason for hiding this comment

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

lfg

@danirabbit danirabbit merged commit d6ef545 into master Apr 20, 2020
Settings Search automation moved this from In progress to Done Apr 20, 2020
@danirabbit danirabbit deleted the searchview branch April 20, 2020 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Waiting for input from the UX team
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Search results look awkward Use application menu's search pattern
2 participants