Skip to content
This repository was archived by the owner on May 20, 2023. It is now read-only.

Commit 21e4b25

Browse files
rkjnshahan
authored andcommitted
Remove visibility.all from @component annotations. This is potentially BREAKING
change. Please review carefully. Breakage will only be caught if the code is covered by tests. Reject this change if you believe it breaks you. If the component needs to be injected anywhere in the app, you can get a runtime exception after this change. This should be caught by tests using your components, and fixing the issue should be trivial, as the exceptions have good error messages, such as "No provider found for FooComponent". PiperOrigin-RevId: 190967017
1 parent 8083766 commit 21e4b25

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

lib/material_datepicker/next_prev_buttons.dart

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,6 @@ export 'package:angular_components/src/material_datepicker/sequential.dart';
4646
''',
4747
styleUrls: const ['next_prev_buttons.scss.css'],
4848
directives: const [GlyphComponent],
49-
// TODO(google): Change to `Visibility.local` to reduce code size.
50-
visibility: Visibility.all,
5149
)
5250
class NextPrevComponent implements OnDestroy {
5351
final ChangeDetectorRef _changeDetector;

0 commit comments

Comments
 (0)