Skip to content

Search not working on mobile #8712

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
tttapa opened this issue Aug 12, 2021 · 5 comments
Closed

Search not working on mobile #8712

tttapa opened this issue Aug 12, 2021 · 5 comments

Comments

@tttapa
Copy link
Contributor

tttapa commented Aug 12, 2021

Describe the bug
When tapping the search field in a mobile browser, it immediately loses focus and it is impossible to type a query.

Screenshots
In the following recordings, I'm just tapping the search field:

1.8.20 works fine (Chrome):

chrome-1.8.20.doxygen.mp4

1.9.2 (master) doesn't work (Chrome):

chrome-1.9.2.doxygen.mp4

1.9.2 (master) doesn't work (Firefox):

firefox-1.9.2.doxygen.mp4

To Reproduce

cat > header.h << EOF
/** @brief Some class. */
class C {};
EOF
doxygen -g
doxygen

Doxygen-8712.zip

Version
1.9.2 (a4958eb), current master version, compiled from source just now. Ubuntu 20.04 x86_64.

@doxygen
Copy link
Owner

doxygen commented Aug 14, 2021

@tttpa: I've tried on iOS (iPhone) and there the search results window appears ok (a bit too big but that's a different issue).
Can you see if you get any error on the Javascript console? with chrome you can open a new tab and go to URL chrome://inspect to toggle logging on and off.

@tttapa
Copy link
Contributor Author

tttapa commented Aug 14, 2021

It does indeed seem to work fine on iOS (both iPhone and iPad).
I tried a different Android phone, and it has the same issues as in the videos above.

I used chrome://inspect, and there are these messages:

[Intervention] Images loaded lazily and replaced with placeholders. Load events are deferred. See https://crbug.com/954323

Screenshot from 2021-08-14 21-50-32

Even if I click the search bar in the remote DevTools on my computer (instead of using the touch screen on my phone), it still doesn't work.

@doxygen
Copy link
Owner

doxygen commented Aug 16, 2021

@tttapa Please verify if the problem is solved with the referenced commit. It is a bit hard for me to test as I don't own an Android mobile device. Only tested with the simulator that comes with Android Studio.

@doxygen doxygen added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 16, 2021
@doxygen
Copy link
Owner

doxygen commented Aug 18, 2021

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.2.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Aug 18, 2021
@doxygen doxygen closed this as completed Aug 18, 2021
@tttapa
Copy link
Contributor Author

tttapa commented Aug 30, 2021

Please verify if the problem is solved with the referenced commit. It is a bit hard for me to test as I don't own an Android mobile device. Only tested with the simulator that comes with Android Studio.

That does indeed seem to fix it, it works now using version 1.9.2, thanks a lot!

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

No branches or pull requests

2 participants