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

Fixes #22540 - auto-focus search bar #5246

Merged
merged 1 commit into from Feb 14, 2018

Conversation

sean797
Copy link
Member

@sean797 sean797 commented Feb 9, 2018

This is somewhat subjective, I'm not sure how everyone feels about it

@timogoebel you mentioned it currently takes too many clicks to find a host, please try this out and let us know your thoughts.

@Rohoover you may also have an opinion on this.

@theforeman-bot
Copy link
Member

Issues: #22540

@timogoebel
Copy link
Member

@sean797: Thanks for putting effort into this, but I'm afraid this doesn't really solve my issue.

What I think, we need is a global search bar with typeahead. Something like this:

image

I think we need something that is visible on every page and that allows you to navigate to a host just by typing the hostname.

@ShimShtein
Copy link
Member

@timogoebel, I think I have something for you: https://github.com/ShimShtein/foreman-opensearch. It adds the ability to search foreman from browser's searchbox/omnibox.

Copy link
Member

@dLobatog dLobatog left a comment

Choose a reason for hiding this comment

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

This might not be of everyone's liking, but personally I think it makes sense. 👍

Let's wait in case someone says otherwise

Copy link

@Rohoover Rohoover left a comment

Choose a reason for hiding this comment

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

I know that there have been talks about a global search but I don't know what the status of it is at this time. I think it could be valuable for users who want to get to something specific but don't want to wade into the UI.

Copy link
Member

@lzap lzap left a comment

Choose a reason for hiding this comment

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

I don't like it. I LOVE IT!

@ohadlevy
Copy link
Member

ohadlevy commented Feb 13, 2018 via email

Copy link
Member

@dLobatog dLobatog left a comment

Choose a reason for hiding this comment

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

@ohadlevy I don't think it's critical to test - from a quick check, a few forms have 'name' as autofocus, but create forms would not have a search bar in general. We can fix any of these anecdotes later if needed/

@dLobatog dLobatog merged commit 45f16ec into theforeman:develop Feb 14, 2018
@dLobatog
Copy link
Member

Thanks @sean797 !

@sean797 sean797 deleted the search-focus branch February 14, 2018 10:02
@lzap
Copy link
Member

lzap commented Feb 15, 2018

I see one misbehavior - when I load a page and it is properly auto-focus and I switch to a different browser tab and go back, auto-suggestion drop down opens up without typing or anything.

@ares
Copy link
Member

ares commented Feb 16, 2018

this happens to me on every new page I enter, Chrome 63, I hope I'm the only one since this is pretty disturbing

@sean797
Copy link
Member Author

sean797 commented Feb 20, 2018

I'm not sure what we can or should do about that? Feel free to revert if no one has a good solution.

@ohadlevy
Copy link
Member

/cc @waldenraines @sharvit @amirfefer

@ohadlevy
Copy link
Member

👍 to revert, this makes a horrible ux :(

@sean797
Copy link
Member Author

sean797 commented Feb 21, 2018

Created #5266

@timogoebel
Copy link
Member

@sean797: Thanks anyway for your attempt. This is a good idea. I think it's the small things that make a large difference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
9 participants