Skip to content

Use full page for search results #2319

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

Closed
wants to merge 1 commit into from

Conversation

wilzbach
Copy link
Member

@wilzbach wilzbach commented Mar 31, 2018

A start and experiment with using the full site for showing the search results.

Preview:

image

This is the initial "proof of concept". When I find more time, I will try to tweak it and make it more "visually appealling". Ideas or examples of other good searches are of course always welcome.

See also:

BTW in case someone wants to test this locally, the most pleasant way is with:

make -f posix.mak apidocs-serve

(as then the JS/CSS is always "up-to-date" / needs to manual copy-overs)

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@wilzbach wilzbach requested a review from CyberShadow as a code owner March 31, 2018 05:40
@dlang-bot dlang-bot added the WIP label Mar 31, 2018
@wilzbach wilzbach requested a review from MartinNowak as a code owner March 31, 2018 05:40
@wilzbach wilzbach removed the WIP label Jun 8, 2018
@wilzbach wilzbach changed the title [WIP] Use full page for search results Use full page for search results Jun 8, 2018
@wilzbach wilzbach force-pushed the full-page-search branch 2 times, most recently from f1b6d40 to 6698777 Compare June 8, 2018 12:09
@wilzbach
Copy link
Member Author

wilzbach commented Jun 8, 2018

Okay. I finally got back to this and while this isn't perfect, imho it's a lot better than the dropdown.

Old:

image

New:

image

What do you think?

@CyberShadow
Copy link
Member

Not sure it's an improvement... it just throws more information in your face, most of which you don't care about, without any organization, and blocks the entire page while doing it. The screenshot in your example isn't of a typical search, I think they will typically be of more than two letters.

@wilzbach
Copy link
Member Author

What do other people think about this?

CC @thewilsonator maybe

@thewilsonator
Copy link
Contributor

The dropdown is dynamically repopulated, correct? Is the new one?

@wilzbach
Copy link
Member Author

The dropdown is dynamically repopulated, correct? Is the new one?

Yes and yes. Just test it out yourself via the DAutoTest preview. Select one page on the diff and enter things in the search bar.

@thewilsonator
Copy link
Contributor

Hmm, I can't seem to trigger either the dropdown on the main site nor the new one on the DAutoTest preview.

@wilzbach
Copy link
Member Author

wilzbach commented Jan 12, 2019

@thewilsonator
Copy link
Contributor

I think I like the older one more, possibly because I'm used to dropdown menus and the new ones is a bit jarring if you're not expecting that to happen, but not by a whole lot.

All the new one does for me is having a slightly bigger font size. If it made use of that space e.g:

  • had the first sentence of the documentation beneath the symbol link, providing a description of the symbol rather than just its name (useful for unfamiliar symbols (std.mathspecial anyone?)),
  • increased the size/usfulness of the icon next to the symbol e.g. I know some of those are e.g classes, but its very hard to see and I'm not sure what some of those icons are supposed to be.

it might have some use over the dropdown.

I note that pressing enter take you to the page for the top result for the partial search, given the apparently random ordering of the results, I'd be all for making it "expand" to the new one with the extra info mentioned above instead.

@wilzbach
Copy link
Member Author

We should probably make the search bar more prominent and similar to e.g. like Rust docs does it:

image

image

https://doc.rust-lang.org/std/env/index.html

@jacob-carlborg
Copy link
Contributor

jacob-carlborg commented Jan 13, 2019

I tried the preview now. I don't like it. It's like a wall of text. It's quite unexpected. It's more difficult for the eyes to have to scan to left and right, that's why newspaper columns are narrow and we prefer code to not be more than 80 characters wide.

@RazvanN7
Copy link
Contributor

@wilzbach Seems like people are reluctant to this. Is it ok to close?

@wilzbach wilzbach closed this Jan 28, 2021
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.

6 participants