React Flicking: some combinations of hanger and anchor prevent moving

[gamtiq]

trafficstars

Description

Usage of some combinations of hanger & anchor prevents ability to focus panels. Nor flicking's next method neither panel's focus method does work. I've found that the following combinations cause the issue:

• hanger - "100%", anchor - "100%"
• hanger - "100%", anchor - "0%" or "0"

Steps to check or reproduce

Use the following code sandbox: https://codesandbox.io/s/react-flicking-hanger-anchor-move-issue-mhxcq Try to click "Focus next" button several times. You will see that panels aren't moved.

[WoodNeck]

Thanks for reporting the issue! I found out that this is a bug in Panel's focus method, which prevents Flicking from moving it.

• https://github.com/naver/egjs-flicking/blob/master/src/components/Panel.ts#L136

Flicking's next and prev method uses Panel's focus, while moveTo is not. So please use moveTo method instead while I'm fixing this issue :)

[stale[bot]]

This issue/PR has been automatically marked as stale because it has not had any update (including commits, comments, labels, milestones, etc) for 30 days. It will be closed automatically if no further update occurs in 7 day. Thank you for your contributions!

한글

이 이슈/PR은 commits, comments, labels, milestones 등 30일간 활동이 없어 자동으로 stale 상태로 전환되었습니다. 이후 7일간 활동이 없다면 자동으로 닫힐 예정입니다. 프로젝트 개선에 기여해주셔서 감사합니다.

[gamtiq]

To prevent auto-closing: the issue still takes place in latest version 3.5.1. See the demo: https://codesandbox.io/s/react-flicking-hanger-anchor-move-issue-mhxcq