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

Improved search #76

Closed
wants to merge 4 commits into from
Closed

Improved search #76

wants to merge 4 commits into from

Conversation

Feichtmeier
Copy link
Member

  • move the search dialog atop of the left pane
  • remove the close button

okokok

@jpnurmi @mivoligo since you both were not 100% happy with the search dialog - what about this instead?

- move the search dialog atop of the left pane
- remove the close button
@Jupi007
Copy link
Collaborator

Jupi007 commented Sep 18, 2021

This is way much better than the floating popup 👍

@mivoligo
Copy link
Collaborator

I like this way much more ❤️

@Feichtmeier
Copy link
Member Author

light

light theme

@jpnurmi
Copy link
Contributor

jpnurmi commented Sep 18, 2021

Getting better. 👍 Have you tried with a transparent barrier color to make it look less modal?

@jpnurmi
Copy link
Contributor

jpnurmi commented Sep 18, 2021

Could it be an Autocomplete widget? Aren't those commonly used for searching? 🙂

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Sep 18, 2021

Getting better. +1 Have you tried with a transparent barrier color to make it look less modal?

Well I wanted to avoid this design but I know you all want it so here it is :) This is what you guys all prefer, right?

wellwell

I didn't try the autocomplete Widget yet

@mivoligo
Copy link
Collaborator

Again, we don't have a way to cancel the search using mouse without selecting an option. :)

@Feichtmeier
Copy link
Member Author

Again, we don't have a way to cancel the search using mouse without selecting an option. :)

You can press ESC

@mivoligo
Copy link
Collaborator

I know but for that I would need to use my keyboard 😃

@Feichtmeier
Copy link
Member Author

hehe ^^ well when you press the search button you need to use the keyboard because otherwise you can not type in the letters for search :)

@mivoligo
Copy link
Collaborator

hehe ^^ well when you press the search button you need to use the keyboard because otherwise you can not type in the letters for search :)

That's fair but what if I just clicked the search button out of curiosity. Maybe I was thinking it's a lens I can grab and move around 😉

@Feichtmeier
Copy link
Member Author

Peek 2021-09-19 19-32

@mivoligo for some reason the barrier dismiss does not work 🤷
must be something in that complex dialog hierarchy - since you guys like this design let's get it in and fix the barrier in another PR, ok?

@mivoligo
Copy link
Collaborator

ok 👍

@@ -48,6 +56,100 @@ class MasterPageState extends State<MasterPage> {

filterItems();

void openSearchDialog() {
showDialog(
barrierDismissible: true,
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's true by default so we can omit this.

void openSearchDialog() {
showDialog(
barrierDismissible: true,
barrierColor: Colors.white.withOpacity(0),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe better to use Colors.transparent

@mivoligo
Copy link
Collaborator

It looks a bit weird in the narrow view:
Selection_002

@Feichtmeier
Copy link
Member Author

Okay this sucks this need to be made better, with less code duplication and with the new navigtion I soon push, closing and re-opening the search issue =)

@Feichtmeier Feichtmeier deleted the bettersearch branch September 23, 2021 10:51
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.

None yet

4 participants