Repository navigation
Proposal to add EdgeInsetsDirectional.copyWith #137475
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
While trying to make my app RTL-aware, I noticed that it is not possible to simply swap out EdgeInsets with EdgeInsetsDirectional as it lacks the copyWith method, which is necessary for immutable data structures like this one.
Proposal
In flutter/packages/flutter/lib/src/painting/edge_insets.dart
/* ... */
class EdgeInsetsDirectional extends EdgeInsetsGeometry {
/* ... */
/// Creates a copy of this EdgeInsetsDirectional but with the given
/// fields replaced with the new values.
EdgeInsetsDirectional copyWith({
double? start,
double? top,
double? end,
double? bottom,
}) {
return EdgeInsetsDirectional.only(
start: start ?? this.start,
top: top ?? this.top,
end: end ?? this.end,
bottom: bottom ?? this.bottom,
);
}
/* ... */
}
/* ... */
Activity
[-]EdgeInsetsDirectional.copyWith[/-][+]Proposal to add `EdgeInsetsDirectional.copyWith`[/+]Added EdgeInsetsDirectional.copyWith (#137559)
github-actions commentedon Nov 15, 2023
This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of
flutter doctor -vand a minimal reproduction of the issue.