Improved screenshot taking policy in regards to screen loading period

CHANGELOG.md

@@ -4,6 +4,7 @@
 - **BREAKING**: `Detector.hasGlobalScope` renamed to `Detector.cumulative`
 - **FIX**: popup interactions included in outputs from below them
 - **FIX**: background could be captured before the route transition animation has ended
+- **NEW**: improved screenshot taking policy in regards to screen loading period
 
 # 0.3.3
 

README.md

@@ -7,6 +7,7 @@
   <a href="https://pub.dev/packages/round_spot"><img src="https://img.shields.io/pub/v/round_spot.svg?color=blueviolet" alt="Pub"></a>
   <a href="https://github.com/stasgora/round-spot/actions"><img src="https://github.com/stasgora/round-spot/workflows/build/badge.svg" alt="build"></a>
   <a href="https://github.com/tenhobi/effective_dart"><img src="https://img.shields.io/badge/style-effective_dart-40c4ff.svg" alt="style: effective dart"></a>
+  <a href="https://pub.dev/packages/round_spot/score"><img src="https://badges.bar/round_spot/likes" alt="likes"></a>
   <a href="https://github.com/stasgora/round-spot/blob/master/LICENSE"><img src="https://img.shields.io/badge/license-MIT-blue.svg" alt="License: MIT"></a>
 
 <img src="https://raw.githubusercontent.com/stasgora/round-spot/master/assets/banner.png" style="width: 100%" alt="Banner"><br>

lib/src/components/background_manager.dart

@@ -31,23 +31,21 @@ class BackgroundManager {
 
   /// Determines if its necessary to take a screenshot when event is recorded
   void onEvent(Offset event, Session session, GlobalKey areaKey) {
-    if (session.background == null || _eventOutsideScreenshot(event, session)) {
+    var baseCheck = session.background == null || !session.scrollable;
+    if (baseCheck || _eventOutsideScreenshot(event, session)) {
       _takeScreenshot(session, areaKey);
     }
   }
 
   bool _eventOutsideScreenshot(Offset event, Session session) {
-    if (!session.scrollable) return false;
-    var offset = session.backgroundOffset;
-    return !(offset & session.bgSize!).contains(event);
+    return !(session.backgroundOffset & session.bgSize!).contains(event);
   }
 
   /// Captures a screenshot from a [RepaintBoundary] using its [GlobalKey]
   /// It than joins it with the already assembled image
   /// replacing the part that's underneath it
   void _takeScreenshot(Session session, GlobalKey areaKey) async {
     if (!session.scrollable) {
-      if (session.background != null) return;
       session.background = await _captureImage(areaKey, session.pixelRatio);
       return;
     }