-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Mobile: scrolling to the bottom and opening navigation causes the view to jump #37
Comments
Thanks, this is a good bug report, it is detailed and reproducible, and I will take the time to deal with it. |
This may be an issue with animation speed settings in Android devtools... I am on .5 and it doesn't seem to "jump" |
@Brian-Pho After testing, I found that this issue only occurs in device emulators for Chrome and Opera devtools; the simulator will automatically change the value of |
I just tested it on my physical Pixel 3 device and it jumps. If you aren't seeing it anywhere else, feel free to ignore it. But if you have the fix please let me know because I'm using this theme for one of my websites. Thanks. |
What's the type and version of broswer on your Pixel 3 device? |
I'm using Chrome version 80.0.3987.149 on Android 10, Pixel 3 Build/QQ1A.191205.008 |
- Header 및 Index Page 반응형 스타일 개선 - install mobile-detect package - 모바일 환경에서 Header를 숨기지 않고 항상 표시하도록 구현
@Brian-Pho |
I don't have a pixel 3 either.... Only a 2xl
…On Thu, Apr 16, 2020, 12:39 PM Cotes ***@***.***> wrote:
@Brian-Pho <https://github.com/Brian-Pho>
Sorry for keeping you waiting so long. Now I have tried to fix this issue
at 2395a34
<2395a34>
(it passed the test on Chrome devTool), but I don't have a physical Pixel
3. Could you be a volunteer to help me test whether this compatibility
problem is solved or not?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOXMXW3MBC2K5SJUZR4ZIULRM4YFLANCNFSM4K7OH66A>
.
|
If the Pixel 2 XL runs Android 10 & chrome 80, it should be enough to test the problem. P.S. I don't have any android devices, so I can't test it. |
My apologies for not responding. It is difficult to test the fix since this isn't deployed anywhere. I'll test it by applying the fix to my own site and commenting on my results. |
It has been deployed to the demo site https://chirpy.cotes.info |
I can confirm the fix works on my Pixel 3, thanks for fixing. :) |
Describe the bug
The bug is unexpected behaviour when the user is on mobile and returns from navigation.
To Reproduce
Expected behavior
The view should not jump after going to navigation.
Screenshots
Desktop:
Smartphone:
Notes
This bug doesn't occur anywhere else on the page except when at the very bottom.
The text was updated successfully, but these errors were encountered: