-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
UI improvements Planning Issue : Searchbar and Search Results page #5708
Comments
Hi! This is looking great. I think more is coming via @ebarry but just wanted to note this great search results page design by @IshaGupta18 here: We have a page in the shared mockups presentation for search results UI where @ebarry and I built on that design idea a bit too: This is all a bit more to do with displaying the results rather than the search input form but is still related. Thank you!!! |
Also noting relation to #5719! |
HI @jywarren |
I think we've worried about the redundancy of the navbar search with the
in-page search box. What do you think is a good way forward?
…On Tue, May 14, 2019 at 10:53 AM Lekhika Dugtal ***@***.***> wrote:
HI @jywarren <https://github.com/jywarren>
The design of search results UI looks great.
I haven been thinking about results of search, and had similar design UI
for that.
I've updated the issue regarding search results UI and added todos
checklists.
What I am curious about the searhbar.
Is there gonna be a new minimal search UI like google or we'll have a
searchbar in navbar only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5708?email_source=notifications&email_token=AAAF6J7PJSLITE2A6SHZWNDPVLG4RA5CNFSM4HMITEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLXUKY#issuecomment-492272171>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAF6J6LJ4QNIMMQO4NZWULPVLG4RANCNFSM4HMITELQ>
.
|
Thank you for the appreciation!
On Tue, May 14, 2019, 10:28 PM Jeffrey Warren <notifications@github.com>
wrote:
… I think we've worried about the redundancy of the navbar search with the
in-page search box. What do you think is a good way forward?
On Tue, May 14, 2019 at 10:53 AM Lekhika Dugtal ***@***.***>
wrote:
> HI @jywarren <https://github.com/jywarren>
> The design of search results UI looks great.
> I haven been thinking about results of search, and had similar design UI
> for that.
> I've updated the issue regarding search results UI and added todos
> checklists.
> What I am curious about the searhbar.
> Is there gonna be a new minimal search UI like google or we'll have a
> searchbar in navbar only.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <
#5708?email_source=notifications&email_token=AAAF6J7PJSLITE2A6SHZWNDPVLG4RA5CNFSM4HMITEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLXUKY#issuecomment-492272171
>,
> or mute the thread
> <
https://github.com/notifications/unsubscribe-auth/AAAF6J6LJ4QNIMMQO4NZWULPVLG4RANCNFSM4HMITELQ
>
> .
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5708?email_source=notifications&email_token=AJXHQZ34YJEKTJ3RT34WI7TPVLVUHA5CNFSM4HMITEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVMDZ4A#issuecomment-492322032>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJXHQZ2BWLEECIIR7LMLA5DPVLVUHANCNFSM4HMITELQ>
.
|
Hi @jywarren , |
Hi @gautamig54 |
@CleverFool77 I am done with my exams. Let's start with the UI implementation. Let's take it one by one. I will get started with the people's page. Thanks! |
Hi @gautmig54 How about we take two issues together. Both people section and questions page sideways. What do you think? |
Hi @jywarren @gauravano Is pill size correct or is there need to chnage this. |
@CleverFool77 I think it will look better if you can reduce the pill size and shift the left column downwards. |
Hi @gautamig54 So should it be 3/4 of what it is right now ? |
Hey this looks pretty great! |
@CleverFool77 The original width of the pill looks better. Maybe you can try altering the height of the pills and also centre align the text in the them. |
I was trying out the code, So I wanted to ask instead of search box, I should have |
Hmm, can you show a quick mock-up? I was hoping Google search would seem
like the "default" to people for the time being. I think it's a little odd
that navbar search would still exist but we could see how it works... it'll
offer tag search results and then offer to continue in the Google search
with the "view all results" entry, so even if it's a little odd I think
people will still be routed eventually back to the full Google search input
once they interact with it.
…On Thu, Jun 6, 2019, 3:55 AM Lekhika Dugtal ***@***.***> wrote:
Hi @ebarry <https://github.com/ebarry> @jywarren
<https://github.com/jywarren>
I was trying out the code, So I wanted to ask instead of search box, I
should have Try with Google search link which would have action of google
search (Similar to UI which you posted above ) ? And the other results for intra
search would be hidden when google search results are shown ?
[image: Screenshot from 2019-06-06 16-20-59]
<https://user-images.githubusercontent.com/26685258/59027624-22551d80-8877-11e9-846e-1a377a6093a7.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5708?email_source=notifications&email_token=AAAF6J7ZPODP22MJCYYG4TDPZDUGDA5CNFSM4HMITEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCPMLI#issuecomment-499447341>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAF6J3LK45HDUR3TJPDRUDPZDUGDANCNFSM4HMITELQ>
.
|
Hi @jywarren |
Oh I see, sorry! So, let's try to not show "Google" unless it is a lot of work to remove it. How does that sound? |
Ohkk!! I'm still pretty confused about the UI , so I'll show the mockup first then let's see and do the further changes. |
Thank you @CleverFool77 !!! |
Hi @gautamig54 Can you look over some of the parts for search section here ? |
Sure! I will take up the # of results and the sort by issues. |
Hi @jywarren I wanted to ask that as per the documentation https://developers.google.com/custom-search/docs/element, the search can be triggered either through searchbox or url in programmatic execution . So we need to execute the |
Yes, and I think that's what we're doing via the
Sorry i'm not quite sure I follow, what are you looking to do? Trigger the search from JavaScript? |
Would you like to try making the navbar search go to So we'll need:
How does this sound? |
Let's add a few more for the navbar search, if it's OK!
Would become: <ul class="typeahead dropdown-menu" role="listbox">
<li class="dropdown-header">TOPICS</li>
<li class="active"><a class="dropdown-item" href="#" role="option"><span style="font-size:100%" class="badge badge-primary">education</span></a></li> |
Sure thing, @CleverFool77! Can you link me to the code? |
I was using the code from the links by @jywarren in the method I wrote. I tried from api documentation to find a method of triggering search without search box and link. |
Hello, @CleverFool77. I take it that the parameter search implementation is what's causing the problem here? If that's the case, how about attaching an What do you think? |
Thank you so much @rexagod |
Done here!!! Yay! |
AMAZING!!!!!!!! The new Search Bar is working great and it's so fun to use!
--
+1 336-269-1539 / @lizbarry <http://twitter.com/lizbarry> / lizbarry.net
…On Mon, Aug 26, 2019 at 4:29 PM Jeffrey Warren ***@***.***> wrote:
Closed #5708 <#5708>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#5708?email_source=notifications&email_token=AABHNH6TUSSPDHQCF4S67I3QGQ4JHA5CNFSM4HMITEL2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTIJJGYI#event-2584908641>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABHNH6T6H7Y724S6NKY2Y3QGQ4JHANCNFSM4HMITELQ>
.
|
New Searhbar ?
Hi @jywarren Regarding this, I wanted to ask that What would be new design of searhbar.
Should it be something like below ?
cc : @gautamig54
Checklists
Searchbar
Search Results
two column @CleverFool77
Pill or Tab design in left column to choose between
search by
options @CleverFool77Sort By date option @gautamig54
try the search by google option and display results of google search @CleverFool77
showing "featured results" for search terms (and thinking of how we can store them)
showing "related tags" in the top of the search results
showing the # of results per type
make the navbar search go to
/search?q=____
instead of/search/_____
replacing the main
Content
search view with Google search results currently stored in wiki page at https://publiclab.org/w/searchadding related tags based on the query
adding a listener so that as you type new queries, it updates related tags (this will be harder but I have some code for it, and we can do in follow-up!)
let's show a spinner icon while it loads the autocomplete results! (as shown below) - say, replacing the magnifying glass icon?
Let's change the inline heading "Tags" to "Topics"
let's put the actual tagname in a badge so it's recognizable as a topic...?
can we filter out powertags from the tags autocomplete
The text was updated successfully, but these errors were encountered: