Skip to content

Commit 7fc8839

Browse files
committed
style(headlines-search): update TODO comments and improve code consistency
- Replace generic 'user' with my username 'fulleni' in TODO comments - Improve consistency of TODO comment formatting - Remove unnecessary comments related to content type filtering - Adjust comments about localization to reflect current implementation status
1 parent a6afbc0 commit 7fc8839

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

lib/headlines-search/view/headlines_search_page.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ class _HeadlinesSearchViewState extends State<_HeadlinesSearchView> {
5656
_showClearButton = _textController.text.isNotEmpty;
5757
});
5858
});
59-
// TODO(user): This logic might need adjustment if not all ContentType values are searchable.
59+
// TODO(fulleni): This logic might need adjustment if not all ContentType values are searchable.
6060
// For now, we default to headline if the current selection is not in the allowed list.
6161
final searchableTypes = [
6262
ContentType.headline,
@@ -110,7 +110,7 @@ class _HeadlinesSearchViewState extends State<_HeadlinesSearchView> {
110110
final textTheme = theme.textTheme;
111111
final appBarTheme = theme.appBarTheme;
112112

113-
// TODO(user): Replace this with a filtered list of searchable content types.
113+
// TODO(fulleni): Replace this with a filtered list of searchable content types.
114114
final availableSearchModelTypes = [
115115
ContentType.headline,
116116
ContentType.topic,
@@ -159,7 +159,7 @@ class _HeadlinesSearchViewState extends State<_HeadlinesSearchView> {
159159
appBarTheme.iconTheme?.color ??
160160
colorScheme.onSurfaceVariant,
161161
),
162-
// TODO(user): Use the new localization extension here.
162+
// TODO(fulleni): Use the new localization extension here.
163163
items: availableSearchModelTypes.map((ContentType type) {
164164
return DropdownMenuItem<ContentType>(
165165
value: type,
@@ -187,7 +187,7 @@ class _HeadlinesSearchViewState extends State<_HeadlinesSearchView> {
187187
controller: _textController,
188188
style: appBarTheme.titleTextStyle ?? textTheme.titleMedium,
189189
decoration: InputDecoration(
190-
// TODO(user): Create a similar localization extension for hint text.
190+
// TODO(fulleni): Create a similar localization extension for hint text.
191191
hintText: 'Search...',
192192
hintStyle: textTheme.bodyMedium?.copyWith(
193193
color:

0 commit comments

Comments
 (0)