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

Enhancements Autosuggest Darstellung und Mobile Funktion #221

Closed
dtpop opened this issue Nov 1, 2019 · 3 comments · Fixed by #385
Closed

Enhancements Autosuggest Darstellung und Mobile Funktion #221

dtpop opened this issue Nov 1, 2019 · 3 comments · Fixed by #385

Comments

@dtpop
Copy link
Member

dtpop commented Nov 1, 2019

Im js vom Autosuggest Plugin gibt es ein keypress. Das ist deprecated und funktioniert nicht auf allen mobilen Devices. Eine Änderung auf keyup löst das Problem.

Die Anzeige des Autosuggest erfolgt standardmäßig im Body und wird dann positioniert.
Eine Änderung auf

m.addClass(s.resultsClass).appendTo(C.closest('.search_it-form')), l(), e(window).load(l).resize(l), C.blur(function() {
            setTimeout(function() {
                m.hide()
            }, 200)
        });

Setzt das Autosuggest in das Suchformular mit dem Input selbst. Es lässt sich dort leichter positionieren. Nachteil: IE kennt kein closest, dort funktioniert es dann nicht mehr.

@tyrant88
Copy link
Member

tyrant88 commented Nov 7, 2019

Das mit keyup würde ich unterstützen.
Das andere empfände ich als Rückschritt, wenn es in weniger Browsern gänzlich funktioniert, um in einigen Browsern eine Verbesserung zu haben.

@alxndr-w
Copy link
Member

@rkemmere du hattest zuletzt das autocomplete-Plugin verbessert gemäß der Commits hier auf GitHub. Nutzt du es noch? Wie würdest du das Problem lösen?

@rkemmere
Copy link
Member

@alexplusde ich schau mir die Problematik mal an. Generell hab ich kein Problem im mobile.

alxndr-w added a commit that referenced this issue Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants