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

Fix CacheSnapshot structure and reduce its size #311

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

inokawa
Copy link
Owner

@inokawa inokawa commented Jan 3, 2024

Removed some fields from snapshot because they are restorable from item sizes.

  • is it better including scrollOffset in snapshot?

@inokawa inokawa force-pushed the cache-snapshot-structure branch 6 times, most recently from 8ec7816 to 29cef37 Compare January 3, 2024 06:04
@inokawa inokawa force-pushed the main branch 2 times, most recently from a74242d to 173e5e2 Compare January 13, 2024 05:12
@inokawa inokawa force-pushed the main branch 2 times, most recently from 5a37173 to 565e251 Compare February 5, 2024 13:00
@inokawa inokawa force-pushed the main branch 2 times, most recently from 9c80325 to 0b752c9 Compare February 15, 2024 12:32
@inokawa inokawa force-pushed the cache-snapshot-structure branch from 29cef37 to 00b137b Compare February 15, 2024 14:24
@inokawa inokawa force-pushed the main branch 5 times, most recently from 1874867 to d329796 Compare February 22, 2024 11:29
@inokawa inokawa force-pushed the cache-snapshot-structure branch 2 times, most recently from f2c1aca to faace19 Compare February 25, 2024 09:55
@inokawa inokawa changed the title Fix CacheSnapshot structure Fix CacheSnapshot structure and reduce its size Feb 25, 2024
@inokawa inokawa force-pushed the cache-snapshot-structure branch 2 times, most recently from 4dc89c8 to f2e962c Compare February 25, 2024 11:21
@inokawa inokawa marked this pull request as ready for review February 25, 2024 11:21
@inokawa inokawa force-pushed the cache-snapshot-structure branch from f2e962c to 4ee04fd Compare February 25, 2024 11:22
@inokawa inokawa force-pushed the cache-snapshot-structure branch from 4ee04fd to 5f733e8 Compare February 25, 2024 11:42
@inokawa inokawa merged commit 0ba1820 into main Feb 25, 2024
4 checks passed
@inokawa inokawa deleted the cache-snapshot-structure branch February 25, 2024 11:55
takecchi referenced this pull request in cuculus-dev/cuculus Feb 29, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [virtua](https://github.com/inokawa/virtua) | [`0.27.5` ->
`0.28.0`](https://renovatebot.com/diffs/npm/virtua/0.27.5/0.28.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/virtua/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/virtua/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/virtua/0.27.5/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/virtua/0.27.5/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

@coderabbitai: ignore

---

### Release Notes

<details>
<summary>inokawa/virtua (virtua)</summary>

### [`v0.28.0`](https://github.com/inokawa/virtua/releases/tag/0.28.0)

[Compare
Source](https://github.com/inokawa/virtua/compare/0.27.5...0.28.0)

-   **BREAKING CHANGE**: The methods of VGridHandle were changed.

#### What's Changed

- Fix CacheSnapshot structure and reduce its size by
[@&#8203;inokawa](https://github.com/inokawa) in
[https://github.com/inokawa/virtua/pull/311](https://github.com/inokawa/virtua/pull/311)
- Add ssrCount prop to Vue Virtualizer by
[@&#8203;inokawa](https://github.com/inokawa) in
[https://github.com/inokawa/virtua/pull/397](https://github.com/inokawa/virtua/pull/397)
- Fix VGridHandle by [@&#8203;inokawa](https://github.com/inokawa) in
[https://github.com/inokawa/virtua/pull/395](https://github.com/inokawa/virtua/pull/395)

**Full Changelog**:
inokawa/virtua@0.27.5...0.28.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/cuculus-dev/cuculus).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMTIuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIxMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant