Skip to content
This repository was archived by the owner on Jan 26, 2026. It is now read-only.

Fix freezing and crashing - #15

Merged
LegendAF merged 4 commits into
mainfrom
tickets/FL-1829
Nov 27, 2025
Merged

LegendAF merged 4 commits into
mainfrom
tickets/FL-1829

Conversation

@josh-flyclops

Copy link
Copy Markdown
Contributor

There were 2 situations where an endless loop would occur. This happens when the bottom of scrolling is reached and we want to load more:

  • When the selected tab gets changed while we try to load more gifs.
  • When there are no more results to load and the end has been reached.

…f scrolling is reached and we want to load more: 1- When the selected tab gets changed during loading. 2- When there are no more results to load and the end has been reached.
@coveralls-official

coveralls-official Bot commented Nov 26, 2025 •

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 19746574386

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 57.547%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/src/components/tab_view.dart 0 1 0.0%
Files with Coverage Reduction New Missed Lines %
lib/src/components/tab_view.dart 1 24.83%
Totals Coverage Status
Change from base Build 19153161071: -0.1%
Covered Lines: 305
Relevant Lines: 530

💛 - Coveralls

@LegendAF LegendAF left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch

@LegendAF
LegendAF merged commit f0312be into main Nov 27, 2025
1 check passed
@LegendAF
LegendAF deleted the tickets/FL-1829 branch November 27, 2025 19:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants