-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: new featureNothing broken; request for a new capabilityNothing broken; request for a new capabilityc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages teamtriaged-designTriaged by Design Languages teamTriaged by Design Languages team
Description
Is there an existing issue for this?
- I have searched the existing issues
- I have read the guide to filing a bug
Use case
There is a default value when isScrollControlled is false for sheets:
flutter/packages/flutter/lib/src/material/bottom_sheet.dart
Lines 598 to 600 in 14c04cd
| maxHeight: isScrollControlled | |
| ? constraints.maxHeight | |
| : constraints.maxHeight * 9.0 / 16.0, |
The value is hard-coded and not allowed to be configured. In order to apply custom constraints in the scenario, users must wrap a
ConstrainedBox.
Proposal
Expose the value as a parameter scrollControlDisabledMaxHeightRatio for:
_BottomSheetLayoutWithSizeListener_RenderBottomSheetLayoutWithSizeListener_ModalBottomSheetModalBottomSheetRouteshowModalBottomSheet
Metadata
Metadata
Assignees
Labels
P2Important issues not at the top of the work listImportant issues not at the top of the work listc: new featureNothing broken; request for a new capabilityNothing broken; request for a new capabilityc: proposalA detailed proposal for a change to FlutterA detailed proposal for a change to Flutterf: material designflutter/packages/flutter/material repository.flutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.flutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamOwned by Design Languages teamtriaged-designTriaged by Design Languages teamTriaged by Design Languages team