Skip to content

fix(LogTable): load entries until screen is filled - #2334

Merged
Antreesy merged 3 commits into
masterfrom
fix/2333/load-more-on-tall-screen
Oct 26, 2026
Merged

Antreesy merged 3 commits into
masterfrom
fix/2333/load-more-on-tall-screen

Conversation

@Antreesy

@Antreesy Antreesy commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator
  • Fixes [Bug]: Vertical monitor prevents loading protocol viewer聽#2333
    • IntersectionObserver reports only changes of visibility
    • If loaded entries fit on the screen, the load-more row stays visible and no further entries load.
    • :key="logStore.allEntries.length" will remount the observer after each load, so it reports the current visibility again.
    • On 'visible' - another poll, 'hidden' - postponed until user scrolls down
    • Test config adjusted to properly stub components
2333-after.mp4

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (ClaudeCode:claude-opus-5-5)

IntersectionObserver reports only changes of visibility. If loaded
entries fit on the screen, the load-more row stays visible and no
further entries load. Remount the observer after each load, so it
reports the current visibility again.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Inline @nextcloud/dialogs and @nextcloud/vue in vitest, so components
that import them can be mounted in tests.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the fix/2333/load-more-on-tall-screen branch from d4bde2d to 3522218 Compare October 26, 2026 09:38
@Antreesy

Copy link
Copy Markdown
Collaborator Author

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport fbc9d9e 3522218 to stable35

@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport fbc9d9e 3522218 to stable34

@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport fbc9d9e 3522218 to stable33

@Antreesy
Antreesy merged commit b4df651 into master Oct 26, 2026
27 of 28 checks passed
@Antreesy
Antreesy deleted the fix/2333/load-more-on-tall-screen branch October 26, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Vertical monitor prevents loading protocol viewer

3 participants