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

手机用搜索输入时输入法用不了 #280

Closed
SourceOnce opened this issue Oct 17, 2021 · 20 comments · Fixed by #283
Closed

手机用搜索输入时输入法用不了 #280

SourceOnce opened this issue Oct 17, 2021 · 20 comments · Fixed by #283
Assignees
Labels
bug Something isn't working
Milestone

Comments

@SourceOnce
Copy link

手机上搜索时输入法弹出一下就弹回去了,包括官方文档也是

@SourceOnce SourceOnce added the bug Something isn't working label Oct 17, 2021
@HEIGE-PCloud
Copy link
Owner

能提供浏览器版本与手机系统版本的相关信息吗?

@HEIGE-PCloud HEIGE-PCloud added the needs more info Issue requires more information from poster label Oct 17, 2021
@HEIGE-PCloud
Copy link
Owner

Upload.from.GitHub.for.iOS.MOV

在 iOS Safari 上我没能复现这一问题,搜索栏在 Chrome 的 device emulation 中也工作正常。因此我怀疑是相关浏览器版本与兼容性造成的问题。

@SourceOnce
Copy link
Author

安卓11,浏览器为yandex和edge,以及安卓版chrome

@SourceOnce
Copy link
Author

SourceOnce commented Oct 18, 2021

Screenrecording_20211018_094303.mp4

三个浏览器都是这种情况

@HEIGE-PCloud
Copy link
Owner

这个搜索框就是个 <input type="text" placeholder="{{ .Site.Params.search.placeholder }}" id="search-input-mobile">

我手边的安卓设备上也没能复现这个问题。

能否请你先访问这个网站,并告诉我

image

这个输入框是否工作正常?

能否请你再访问这个网站,并告诉我它的输入框在你的设备上是否工作正常?

@SourceOnce
Copy link
Author

这两个都是正常的。。。。就是doit的会上面那种问题,可能是手机问题😂😂

@HEIGE-PCloud HEIGE-PCloud added the help wanted Extra attention is needed label Oct 18, 2021
@HEIGE-PCloud
Copy link
Owner

也许换一个输入法试试看?

@HEIGE-PCloud
Copy link
Owner

也许和

$searchInput.addEventListener('focus', () => {
    document.body.classList.add('blur');
    $header.classList.add('open');
}, false);
$searchInput.addEventListener('input', () => {
    if ($searchInput.value === '') $searchClear.style.display = 'none';
    else $searchClear.style.display = 'inline';
}, false);

有关

@SourceOnce
Copy link
Author

代码在哪个文件呀

@HEIGE-PCloud
Copy link
Owner

@SourceOnce
Copy link
Author

苹果设备都可以,我这边的安卓都不可以,不知道什么原因,代码对比loveIt也看不出什么问题。。。。

@HEIGE-PCloud
Copy link
Owner

image

我在 Chrome 和 Firefox 的 emulator 上也复现不出来... 所以还挺麻烦的.......

https://3708120f.hugodoit.pages.dev/ 能否尝试一下这个网址是否工作正常?

@SourceOnce
Copy link
Author

哇,这个是正常的

@SourceOnce
Copy link
Author

https://blog.gdgroups.club/
这个是我的

@HEIGE-PCloud
Copy link
Owner

可能与 #135 有关。#278 也与 #135 有关。

@SourceOnce
Copy link
Author

请问我该如何解决这个问题

@HEIGE-PCloud HEIGE-PCloud removed help wanted Extra attention is needed needs more info Issue requires more information from poster labels Oct 20, 2021
@HEIGE-PCloud
Copy link
Owner

这是最后一个正常工作的版本 https://68984a37.hugodoit.pages.dev/
这是第一个损坏版本 https://79b244b3.hugodoit.pages.dev/

这是造成问题的 commit 564cae8

@HEIGE-PCloud
Copy link
Owner

应该和 if (!searchConfig || isMobile && window._searchMobileOnce || !isMobile && window._searchDesktopOnce) return; 有关。

这个接近 1000 行的 theme.js 已经有向屎山发展的趋势了。没有注释也没有单元测试,维护的时候删一点加一点也不知道会搞坏什么东西 :/

@HEIGE-PCloud HEIGE-PCloud linked a pull request Oct 20, 2021 that will close this issue
@HEIGE-PCloud HEIGE-PCloud self-assigned this Oct 20, 2021
@HEIGE-PCloud HEIGE-PCloud added this to the v0.2.13 milestone Oct 20, 2021
@HEIGE-PCloud
Copy link
Owner

https://do-it-git-fix-search-pcloud.vercel.app/

麻烦帮忙测试一下 这个网址上的搜索框是否工作正常?

@SourceOnce
Copy link
Author

工作正常

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants