Zulip Chat Archive

Stream: general

Topic: Search API docs on mobile


Kevin Buzzard (Nov 10 2021 at 23:21):

I just tried to search the API docs on my android mobile; I tried Chrome and duck duck go browsers and I couldn't find elliptic curves by searching in the search box, I just got octopuses.

Kevin Buzzard (Nov 10 2021 at 23:21):

Dioes it just not work on mobile?

Eric Wieser (Nov 10 2021 at 23:28):

I think we're running into https://bugs.chromium.org/p/chromium/issues/detail?id=154571

Yakov Pechersky (Nov 10 2021 at 23:28):

I never got it working on mobile

Eric Wieser (Nov 10 2021 at 23:29):

https://caniuse.com/?search=SharedWorker suggests it works on mobile firefox

Riccardo Brasca (Nov 10 2021 at 23:33):

It works on Firefox for me

Eric Wieser (Nov 10 2021 at 23:37):

It looks like maybe if we used navigator.serviceWorker instead of SharedWorker we'd be ok, but I can't tell whether the semantics are similar enough

Reid Barton (Nov 11 2021 at 00:15):

An octopus is a hyperelliptic curve right?

Adam Topaz (Nov 11 2021 at 00:18):

Yes, genus 8

Adam Topaz (Nov 11 2021 at 00:20):

Works for me on Firefox/android
Screenshot_20211110-172002.png

Kevin Buzzard (Nov 11 2021 at 09:58):

Also works for me on Firefox! At least that's my own problem solved... . When do we get our own app?

Eric Wieser (Nov 11 2021 at 10:32):

Eric Wieser said:

It looks like maybe if we used navigator.serviceWorker instead of SharedWorker we'd be ok, but I can't tell whether the semantics are similar enough

It looks like https://github.com/GoogleChrome/samples/tree/gh-pages/service-worker/post-message would be a good template to follow if someone wants to try doing this. But https://stackoverflow.com/questions/29741922/prevent-service-worker-from-automatically-stopping suggests that the background search job might end up being killed due to inactivity; vs the current behavior where clicking a search result can kill the worker if no other tabs are open
.

Rob Lewis (Nov 11 2021 at 14:55):

I think that any work on this should probably build on doc-gen#131 . There's one remaining issue there, to make the 404 page work with the new search, but honestly we should have fixed and merged that a long time ago

Eric Wieser (Nov 11 2021 at 15:02):

For me the fact that populating the search index with that PR takes 30s rather than 6s (on my machine) is still a concern

Patrick Massot (Nov 11 2021 at 15:03):

I forgot this wasn't merged. It's really a shame.

Rob Lewis (Nov 11 2021 at 15:17):

Eric Wieser said:

For me the fact that populating the search index with that PR takes 30s rather than 6s (on my machine) is still a concern

I wonder what the difference is, I didn't see anywhere close to a 4x slowdown in Chrome.

Eric Wieser (Nov 11 2021 at 15:22):

Well the indexing data is at least 4x bigger, so it doesn't surprise me that much


Last updated: Dec 20 2023 at 11:08 UTC