Skip to content

Conversation

@akhil-ge0rge
Copy link
Contributor

@akhil-ge0rge akhil-ge0rge commented Jan 28, 2026

Description

This PR introduces support for stacSliverFillRemaining by adding a new widget model and its corresponding parser.

Related Issues

Closes #429

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Code refactor
  • Build configuration change
  • Documentation
  • Chore

Summary by CodeRabbit

  • New Features

    • Added support for SliverFillRemaining so UIs can fill remaining scrollable space with configurable scroll/body behavior.
  • Documentation

    • Added comprehensive documentation describing properties, placement, behavior, and usage examples.
  • Examples

    • Added an interactive gallery example and a new home-screen entry to demonstrate SliverFillRemaining in action.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

📝 Walkthrough

Walkthrough

Adds SliverFillRemaining support: model, JSON serialization, parser, widget-type enum entry, service registration, documentation, and a gallery example asset and navigation entry.

Changes

Cohort / File(s) Summary
Core model & serialization
packages/stac_core/lib/widgets/sliver_fill_remaining/stac_sliver_fill_remaining.dart, packages/stac_core/lib/widgets/sliver_fill_remaining/stac_sliver_fill_remaining.g.dart
New StacSliverFillRemaining StacWidget with child, hasScrollBody, fillOverscroll, plus generated fromJson/toJson helpers.
Parser
packages/stac/lib/src/parsers/widgets/stac_sliver_fill_remaining/stac_sliver_fill_remaining_parser.dart
New StacSliverFillRemainingParser that builds a Flutter SliverFillRemaining, parsing child, hasScrollBody (default true), and fillOverscroll (default false).
Registration & exports
packages/stac_core/lib/foundation/specifications/widget_type.dart, packages/stac/lib/src/framework/stac_service.dart, packages/stac_core/lib/widgets/widgets.dart, packages/stac/lib/src/parsers/widgets/widgets.dart
Adds sliverFillRemaining to WidgetType, registers parser in StacService, and exports model/parser from widgets entry points.
Documentation
docs/docs.json, docs/widgets/sliver_fill_remaining.mdx
Adds docs navigation entry and a new MDX page describing the widget, properties, placement, and JSON example.
Gallery example & manifest
examples/stac_gallery/assets/json/home_screen.json, examples/stac_gallery/assets/json/sliver_fill_remaining_example.json
Adds a home-screen tile for "Stac Sliver Fill Remaining" and a JSON example asset demonstrating a sliverFillRemaining with centered text.

Sequence Diagram(s)

mermaid
sequenceDiagram
participant JSON as "Example JSON Asset"
participant Service as "StacService"
participant Parser as "StacSliverFillRemainingParser"
participant Widget as "Flutter SliverFillRemaining"
JSON->>Service: load JSON (type: sliverFillRemaining)
Service->>Parser: select parser & pass model
Parser->>Widget: construct SliverFillRemaining(child, hasScrollBody, fillOverscroll)
Widget->>Widget: integrated into CustomScrollView/render tree

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

  • Potatomonsta

Poem

🐰 Hopping through code in the soft moonbeam,
I stitch a sliver to finish the dream,
A child centered, space snug and merry,
Parser hums, docs sparkle like cherry,
Hooray—this widget fills the scene! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main change: adding a SliverFillRemaining widget, parser, example, and documentation.
Linked Issues check ✅ Passed The PR successfully implements the requirement from issue #429 by creating the StacSliverFillRemaining widget model, parser, and supporting infrastructure.
Out of Scope Changes check ✅ Passed All changes are scoped to implementing SliverFillRemaining support; no unrelated modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@divyanshub024 divyanshub024 merged commit bb58eff into StacDev:dev Jan 28, 2026
5 checks passed
@akhil-ge0rge akhil-ge0rge deleted the feat-sliver-fill-remaining branch January 28, 2026 19:37
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.

feat: create StacWidget implementation of SliverFillRemaining

2 participants