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

Improvement: Move results URL above summary #1816

Open
geckolinux opened this issue Jan 29, 2020 · 8 comments
Open

Improvement: Move results URL above summary #1816

geckolinux opened this issue Jan 29, 2020 · 8 comments

Comments

@geckolinux
Copy link

@geckolinux geckolinux commented Jan 29, 2020

Hi there, thanks a lot for creating SearX.

Would it be possible to add a UI option that moves the placement of the URL above the result summary, or even at the very top of each result? I almost always give priority to the URL when scanning through results, and having it at the bottom of the result makes my eyes jump around more and slows me down. The "legacy" theme does have this the way I like it, but it's missing many of the niceties of the newer themes. It would be nice if this option could be independent of the theme.

Thanks!

@Pofilo

This comment has been minimized.

Copy link
Collaborator

@Pofilo Pofilo commented Jan 30, 2020

Doing this as an option could be overkilled, however, you can customize the theme to put the URL above the summary.
You can do this in the file:
searx/templates/legacy/result_templates/default.html
By interverting lines 2 and 3 (result_title and url).

Is it enough to you ? This solution requires you to run your own instance.

@geckolinux

This comment has been minimized.

Copy link
Author

@geckolinux geckolinux commented Feb 2, 2020

@Pofilo Thanks for the reply. Thus far, I don't run my own instance, so unfortunately not an option. I suspect that moving the URL closer to the top would be the preference of most users though, so possibly the default could be changed.

@geckolinux geckolinux changed the title Option to move URL display above summary Improvement: Move results URL above summary Feb 7, 2020
@geckolinux

This comment has been minimized.

Copy link
Author

@geckolinux geckolinux commented Feb 7, 2020

Looks like the consensus from other search engines is that most users prefer the URL higher up:

Screenshot_20200207_092601

Screenshot_20200207_092615

Screenshot_20200207_092640

Screenshot_20200207_092711

Screenshot_20200207_092728

So I'm simply changing the title of this issue to a request to move the URL higher by default, that should be suitable for most users.

@geckolinux

This comment has been minimized.

Copy link
Author

@geckolinux geckolinux commented Feb 7, 2020

This is for legacy:
7281592

Can't figure out where to change it for the Oscar theme?

@derwerbebriefkasten

This comment has been minimized.

Copy link

@derwerbebriefkasten derwerbebriefkasten commented Feb 8, 2020

for Oscar theme:
if you move the main_results block above the sidebar_result block and change the parameters like this

...
<div class="col-sm-8" id="main_results">
  ...
<div class="col-sm-4" id="sidebar_results">
  ...
...

in
/searx/templates/oscar/results.html
you can get back to the behaviour of version 0.15.0

Works good but I would also prefer results URL above summary as a default.

@geckolinux

This comment has been minimized.

Copy link
Author

@geckolinux geckolinux commented Feb 9, 2020

@derwerbebriefkasten Thanks a lot for the explanation!

So you're saying to move the entire main_results block from line 79 all the way down to 154 above sidebar_results ? I don't understand why the order of the entire results has to change just to move the URL?

Thanks!

@derwerbebriefkasten

This comment has been minimized.

Copy link

@derwerbebriefkasten derwerbebriefkasten commented Feb 9, 2020

@geckolinux Obviously I have given a solution to another request. When I tried your request on my mobile phone, I noticed that all possible infoboxes are now displayed first, which is quite uncomfortable, my description changes this back but has nothing to do with your request.
Sorry for the confusion.
But I support your request, it makes the result list clearer for me.

@dalf

This comment has been minimized.

Copy link
Collaborator

@dalf dalf commented Feb 9, 2020

Out of topic: @derwerbebriefkasten about the infoboxes see #1777

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

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.