batchInsert에서 사용하는 addedElements 배열과 elIdx값의 싱크가 맞지 않습니다.

[hynarin-crema]

Description

안녕하세요.
sync 함수에서 실행되는 batchInsert에 대해 문의드립니다.
diffResult.added가 [0, 2, 3]일 경우, [0, 2]에 대해서 batchInsert함수가 한번 실행되고 [3]에 대해서 batchInsert함수가 한번 더 실행됩니다.

[3]에 대해서 batchInsert함수가 호출될 때 리턴하는 element가 잘못 지정되고 있습니다.
(이때 addedElements는 [0번째 요소, 2번째 요소, 3번째 요소]이고, startIdx는 2입니다.)
elIdx는 항상 0부터 시작하여, element의 값으로 addedElements[2]가 아닌 addedElements[0]이 할당되고 있습니다.
그 결과 addedElements[2]에 동적으로 width가 할당되지 않는 문제가 발생하고 있습니다. 확인 부탁드립니다.

Steps to check or reproduce

sync 함수를 아래와 같이 수정해야 하지 않을까 싶습니다.

...
if (diffResult.added.length > 0) {
    var startIdx_1 = -1;
    var prevIdx_2 = -1;
    var addedElements_1 = rendered.slice(prevList.length);
    diffResult.added.forEach(function (addedIdx, idx) {
      if (startIdx_1 < 0) {
        startIdx_1 = idx;
      }
      if (prevIdx_2 >= 0 && addedIdx !== prevIdx_2 + 1) {
        added.push.apply(added, __spread(batchInsert(renderer, diffResult, addedElements_1, startIdx_1, idx + 1)));
        startIdx_1 = -1;
        prevIdx_2 = -1;
      } else {
        prevIdx_2 = addedIdx;
      }
    });
    if (startIdx_1 >= 0) {
     // 수정한 부분: addedElements_1 -> addedElements_1.slice(startIdx_1)
      added.push.apply(added, __spread(batchInsert(renderer, diffResult, addedElements_1.slice(startIdx_1), startIdx_1)));
    }
...

[malangfox]

안녕하세요 @hynarin-crema 님.

전달 주신 부분을 수정하여 @egjs/flicking 4.11.5-beta.4 버전과 @egjs/react-flicking 4.11.5-beta.4 버전이 배포되었습니다.
위 버전의 패키지를 사용해보시고 겪고 계신 이슈가 해결되었는지 확인이 가능할까요?
만약 다른 프레임워크의 패키지를 사용 중이시라면 해당 프레임워크를 위한 패키지 또한 베타 버전으로 배포하도록 하겠습니다.

감사합니다.

[hynarin-crema]

안녕하세요. @malangfox님. 빠른 확인과 대응 감사합니다. vue 프레임워크를 사용하고 있어, vue에 대한 패키지도 배포 부탁드립니다. 감사합니다.

…

On Wed, Sep 11, 2024 at 4:05 PM malangfox ***@***.***> wrote: 안녕하세요 @hynarin-crema <https://github.com/hynarin-crema> 님. 전달 주신 부분을 수정하여 @egjs/flicking 4.11.5-beta.4 버전과 @egjs/react-flicking 4.11.5-beta.4 버전이 배포되었습니다. 위 버전의 패키지를 사용해보시고 겪고 계신 이슈가 해결되었는지 확인이 가능할까요? 만약 다른 프레임워크의 패키지를 사용 중이시라면 해당 프레임워크를 위한 패키지 또한 베타 버전으로 배포하도록 하겠습니다. 감사합니다. — Reply to this email directly, view it on GitHub <#887 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BAAJOBA6NKQWUSFDFDKBMA3ZV7TR7AVCNFSM6AAAAABNHLR2U2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBSHAZDMNBUG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- * <https://www.cre.ma/> 크리마* *정나린 / 소프트웨어 엔지니어* *E.* ***@***.*** *T.* 1522-0305 *A.* 서울특별시 성동구 아차산로17길 48 SKV1센터 1동 1113호

[malangfox]

@hynarin-crema

@egjs/vue-flicking 및 @egjs/vue3-flicking의 4.11.5-beta.4 버전이 배포되었습니다.
해당 버전의 패키지를 이용해보시면 되겠습니다.

[hynarin-crema]

@malangfox
재현되지 않는 것 확인했습니다. 감사합니다. 👍
v4.11.5에 반영되는 것으로 알고 있으면 될까요? 예상 배포시점은 언제일지 궁금합니다.