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

Bug: Window layout error when creating a snippet copy in the mobile version #516

Open
KseniyaKerzhner opened this issue Jun 20, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@KseniyaKerzhner
Copy link

KseniyaKerzhner commented Jun 20, 2024

Summary

In the mobile version, after creating a copy of the snippet, the location of the window view changes from vertical to horizontal, which is not convenient when used in the mobile version. There is no change view button in the mobile version.

issue.20.06.24.mp4

Reproduction steps

1. User authorized
2. Created a snippet in any language
3. Open the code view mode
4. Press "toggle device toolbar"
4. Select Pixel 7 device
5. Open one of the existing snippets
6. Click the button to copy the snippet
7. Create a copy

Expected result

A copy of the snippet opened and the windows are arranged vertically as in the first snippet

Actual result

Opened a copy of the snippet and the windows are horizontal, can not be changed

Browsers

Chrome

OS

Windows

@KseniyaKerzhner KseniyaKerzhner added the bug Something isn't working label Jun 20, 2024
@KseniyaKerzhner
Copy link
Author

Еще немного потестировав этот баг обнаружила, что лечится переворотом экрана в горизонтальное положение.
Оставить как баг или удалить?

@bobrov-site
Copy link
Contributor

@KseniyaKerzhner думаю такого поведения не должно быть в любом случае. Могу взять задачу в работу

@bobrov-site
Copy link
Contributor

Поскольку отмажки не было, то беру задачу в работу.

@bobrov-site
Copy link
Contributor

Нужен help от фронтов. Я знаю почему такая проблема возникает. Ситуация следующая:

После того, как мы нажимаем на кнопку копировать сниппет, нас перебрасывает на новый url. Соответственно, страница как-будто не перегружается, а значит компонент не рендерится

На странице сниппетов, я ранее создавал хук в котором отслеживаются изменение размеров экрана, но по какой-то причине он не работает

useEffect(() => {
if (isNotMobile) {
dispatch(actions.updateDirection('horizontal'));
} else {
dispatch(actions.updateDirection('vertical'));
}
}, [dispatch, isNotMobile]);

Соотвественно, когда мы открываем с телефона и переходим на новую страницу у нас идет сброс до нашего стэйта

direction: 'horizontal',

Как решить эту проблему?

@KseniyaKerzhner KseniyaKerzhner changed the title Bug: Ошибка расположения окон при создании копии сниппета в мобильной версии Bug: Window layout error when creating a snippet copy in the mobile version Jul 8, 2024
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
Status: No status
Development

No branches or pull requests

2 participants