Skip to content

Refactor visibility_detector to avoid forcing compositing. #367

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

Merged
merged 19 commits into from
May 24, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 29 additions & 14 deletions packages/visibility_detector/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,36 +1,51 @@
# 0.3.3
# CHANGELOG

## 0.4.0

* Refactor to avoid forcing composition in the layer/render trees.
* Remove `VisibilityDetectorLayer`.
* Add `RenderVisibilityDetectorBase` as a mixin that mostly takes over
functionality from the old layer.
* Remove the lookup map/method for getting former screen rects and instead add
the rect to `VisibilityInfo`.

## 0.3.3

* Re-apply Flutter framework bindings' null safety calls but set SDK
constraints correctly to 2.12.0 instead.

# 0.3.2
## 0.3.2

* Reverts change from 0.3.0 where the Flutter version constraint should have
been set to 2.12.0 instead of 2.10.5.

# 0.3.1-dev
## 0.3.1-dev

* Populate the pubspec `repository` field.

# 0.3.0
## 0.3.0

* Move to Flutter version 2.10.5 and update dependencies' null safety calls.

# 0.2.2
## 0.2.2

* Minor internal changes to maintain forward-compatibility with [flutter#91753](https://github.com/flutter/flutter/pull/91753).

# 0.2.1
## 0.2.1

* Bug fix for using VisibilityDetector with FittedBox and Transform.scale [issue #285](https://github.com/google/flutter.widgets/issues/285).

# 0.2.0
## 0.2.0

* Added `SliverVisibilityDetector` to report visibility of `RenderSliver`-based
widgets. Fixes [issue #174](https://github.com/google/flutter.widgets/issues/174).

# 0.2.0-nullsafety.1
## 0.2.0-nullsafety.1

* Revert change to add `VisibilityDetectorController.scheduleNotification`,
which introduced unexpected memory usage.

# 0.2.0-nullsafety.0
## 0.2.0-nullsafety.0

* Update to null safety.

Expand All @@ -41,7 +56,7 @@
* Add `VisibilityDetectorController.scheduleNotification` to force firing a
visibility callback.

# 0.1.5
## 0.1.5

* Compatibility fixes to `demo.dart` for Flutter 1.13.8.

Expand All @@ -52,24 +67,24 @@

* Added a "Known limitations" section to `README.md`.

# 0.1.4
## 0.1.4

* Style and comment adjustments.

* Fix a potential infinite loop in the demo app and add tests for it.

# 0.1.3
## 0.1.3

* Fixed positioning of text selection handles for `EditableText`-based
widgets (e.g. `TextField`, `CupertinoTextField`) when used within a
`VisibilityDetector`.

* Added `VisibilityDetectorController.widgetBoundsFor`.

# 0.1.2
## 0.1.2

* Compatibility fixes for Flutter 1.3.0.

# 0.1.1
## 0.1.1

* Added `VisibilityDetectorController.forget`.
2 changes: 2 additions & 0 deletions packages/visibility_detector/example/lib/main.dart
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,8 @@ class VisibilityDetectorDemo extends StatelessWidget {
Widget build(BuildContext context) {
return MaterialApp(
title: title,
scrollBehavior:
const MaterialScrollBehavior().copyWith(scrollbars: false),
theme: ThemeData(primarySwatch: Colors.blue),
home: VisibilityDetectorDemoPage(key: key, useSlivers: useSlivers),
);
Expand Down

This file was deleted.

Loading