Skip to content

This fixes an exception thrown when using GoRouter - #144

Merged
cnlklink merged 2 commits into
mainfrom
fix-overlay-exception
Sep 25, 2026
Merged

cnlklink merged 2 commits into
mainfrom
fix-overlay-exception

Conversation

@cnlklink

Copy link
Copy Markdown
Contributor

The removes the SelectionArea that is inserted into the widget tree by _RouterApp. Unfortunately this won't work correctly with GoRouter because the SelectionArea ends up as the parent of Overlay with how the router works. Instead the developer needs to insert SelectionArea for each Route their app builds.

@github-actions

Copy link
Copy Markdown

Code Coverage Report - 119 of 353 lines covered ( ⛔ 33.71%)

lib - 119 of 353 lines covered ( ⛔ 33.71%)

lib/flutter_controls_core.dart - 0 of 8 lines covered ( ⛔ 0.00%)

⛔ This file is missing coverage.

lib/src - 119 of 345 lines covered ( ⛔ 34.49%)

lib/src/app_scaffold.dart - 36 of 152 lines covered ( ⛔ 23.68%)

Uncovered lines: ⚠️ 18, 36-37, 46, 51, 56, 58, 61-62, 64, 66-67, 69, 74, 78, 80, 82, 84, 92, 94-95, 97, 99, 101, 103, 105, 107-108, 110, 114, 116-117, 122, 127-128, 130-132, 134, 136-137, 141, 149-150, 152, 154-155, 157, 166, 168-169, 175, 177-178, 181, 183-184, 187, 189, 191, 193-196, 204, 211, 216-218, 221, 224, 226, 229, 231, 242-244, 248, 250, 252-254, 256, 265-270, 278, 280, 282-283, 285-291, 293, 297-298, 300-301, 303-304, 308, 311, 325, 423, 456, 541, 547, 551, 553

lib/src/fermi_theme.dart - 0 of 66 lines covered ( ⛔ 0.00%)

⛔ This file is missing coverage.

lib/src/otel_tracing.dart - 23 of 44 lines covered ( ⛔ 52.27%)

Uncovered lines: ⚠️ 72-81, 103-104, 111, 117, 119, 194, 196, 198, 203, 205-206

lib/src/widgets - 60 of 83 lines covered ( ⛔ 72.29%)

lib/src/widgets/param_panel.dart - 11 of 12 lines covered ( ✅ 91.67%)

Uncovered lines: ⚠️ 40

lib/src/widgets/param_row.dart - 49 of 71 lines covered ( ⛔ 69.01%)

Uncovered lines: ⚠️ 69, 71-72, 76-77, 79-83, 85, 91-92, 96-97, 99-100, 122, 136-138, 146

@mguzman04

Copy link
Copy Markdown
Contributor

That's unfortunate but it makes sense.

Just a minor request, could you up the version in pubspec.yaml The library is using git tags now and allows apps to specify a version instead of pointing to the main branch

@github-actions

Copy link
Copy Markdown

Code Coverage Report - 119 of 353 lines covered ( ⛔ 33.71%)

lib - 119 of 353 lines covered ( ⛔ 33.71%)

lib/flutter_controls_core.dart - 0 of 8 lines covered ( ⛔ 0.00%)

⛔ This file is missing coverage.

lib/src - 119 of 345 lines covered ( ⛔ 34.49%)

lib/src/app_scaffold.dart - 36 of 152 lines covered ( ⛔ 23.68%)

Uncovered lines: ⚠️ 18, 36-37, 46, 51, 56, 58, 61-62, 64, 66-67, 69, 74, 78, 80, 82, 84, 92, 94-95, 97, 99, 101, 103, 105, 107-108, 110, 114, 116-117, 122, 127-128, 130-132, 134, 136-137, 141, 149-150, 152, 154-155, 157, 166, 168-169, 175, 177-178, 181, 183-184, 187, 189, 191, 193-196, 204, 211, 216-218, 221, 224, 226, 229, 231, 242-244, 248, 250, 252-254, 256, 265-270, 278, 280, 282-283, 285-291, 293, 297-298, 300-301, 303-304, 308, 311, 325, 423, 456, 541, 547, 551, 553

lib/src/fermi_theme.dart - 0 of 66 lines covered ( ⛔ 0.00%)

⛔ This file is missing coverage.

lib/src/otel_tracing.dart - 23 of 44 lines covered ( ⛔ 52.27%)

Uncovered lines: ⚠️ 72-81, 103-104, 111, 117, 119, 194, 196, 198, 203, 205-206

lib/src/widgets - 60 of 83 lines covered ( ⛔ 72.29%)

lib/src/widgets/param_panel.dart - 11 of 12 lines covered ( ✅ 91.67%)

Uncovered lines: ⚠️ 40

lib/src/widgets/param_row.dart - 49 of 71 lines covered ( ⛔ 69.01%)

Uncovered lines: ⚠️ 69, 71-72, 76-77, 79-83, 85, 91-92, 96-97, 99-100, 122, 136-138, 146

@mguzman04 mguzman04 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

@cnlklink
cnlklink merged commit 03651c7 into main Sep 25, 2026
2 checks passed
@cnlklink
cnlklink deleted the fix-overlay-exception branch September 25, 2026 18:24
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.

2 participants