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

TypeError: Unable to get property 'slotHeaderSize' of undefined or null reference #247

Closed
conlanpatrek opened this issue Jul 3, 2020 · 1 comment
Labels

Comments

@conlanpatrek
Copy link

conlanpatrek commented Jul 3, 2020

Describe

Seeing this stacktrace in logs:

TypeError: Unable to get property 'slotHeaderSize' of undefined or null reference
  at value(./node_modules/vue-virtual-scroll-list/dist/index.js:229:9)
  at this(./node_modules/vue-virtual-scroll-list/dist/index.js:203:9)
  at this(./node_modules/vue-virtual-scroll-list/dist/index.js:194:11)
  at methods.onScroll(./node_modules/vue-virtual-scroll-list/dist/index.js:824:9)

EDIT: I am running in page mode, here are the details

<virtual-list
      class="leaderboard__container"
      page-mode
      :style="{ minWidth }"
      :data-sources="rows"
      :data-key="row => row.user.id"
      :data-component="Item"
      :extra-props="{
          //...
      }"
      :keeps="100"
      @expand="expanded = $event"
      @selectVideo="$emit('selectVideo', $event)"
/>

To Reproduce

Haven't been able to reproduce myself, but I only see two spots in the codebase where the scroll event might fire while virtual.param is null:

https://github.com/tangbc/vue-virtual-scroll-list/blob/master/src/index.js#L81-L86
-- Please consider unregistering the scroll event before calling virtual.destroy()

https://github.com/tangbc/vue-virtual-scroll-list/blob/master/src/index.js#L186-L190
-- Unlikely to be causing my issue. reset() isn't called in this codebase and we're not using a component ref
-- Please consider installing the new Virtual instance before scrolling, or deregistering scroll handlers during this operation

Reproduce demo

Unable

Other

  • Version 2.3
  • Browser IE11 from what I can see

Thank you

@tangbc
Copy link
Owner

tangbc commented Jul 10, 2020

OK, I will fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants