Skip to content

Commit 5c26056

Browse files
chrisbobbechrisbobbe
committed
compose [nfc]: Shorten {topic,content}Controller to just {topic,content}
1 parent 824c294 commit 5c26056

File tree

4 files changed

+36
-36
lines changed

4 files changed

+36
-36
lines changed

‎lib/widgets/action_sheet.dart

+4-4
Original file line numberDiff line numberDiff line change
@@ -334,15 +334,15 @@ class QuoteAndReplyButton extends MessageActionSheetMenuItemButton {
334334
if (composeBoxController == null) return;
335335
if (
336336
composeBoxController is StreamComposeBoxController
337-
&& composeBoxController.topicController.textNormalized == kNoTopicTopic
337+
&& composeBoxController.topic.textNormalized == kNoTopicTopic
338338
&& message is StreamMessage
339339
) {
340-
composeBoxController.topicController.value = TextEditingValue(text: message.topic);
340+
composeBoxController.topic.value = TextEditingValue(text: message.topic);
341341
}
342342

343343
// This inserts a "[Quoting…]" placeholder into the content input,
344344
// giving the user a form of progress feedback.
345-
final tag = composeBoxController.contentController
345+
final tag = composeBoxController.content
346346
.registerQuoteAndReplyStart(PerAccountStoreWidget.of(pageContext),
347347
message: message,
348348
);
@@ -360,7 +360,7 @@ class QuoteAndReplyButton extends MessageActionSheetMenuItemButton {
360360
// posting permission or when a DM recipient becomes deactivated.
361361
composeBoxController = findMessageListPage().composeBoxController;
362362
if (composeBoxController == null) return;
363-
composeBoxController.contentController
363+
composeBoxController.content
364364
.registerQuoteAndReplyEnd(PerAccountStoreWidget.of(pageContext), tag,
365365
message: message,
366366
rawContent: rawContent,

‎lib/widgets/compose_box.dart

+18-18
Original file line numberDiff line numberDiff line change
@@ -1100,29 +1100,29 @@ class _ComposeBoxLayout extends StatelessWidget {
11001100
}
11011101

11021102
sealed class ComposeBoxController {
1103-
ComposeContentController get contentController => _contentController;
1104-
final _contentController = ComposeContentController();
1103+
ComposeContentController get content => _content;
1104+
final _content = ComposeContentController();
11051105

11061106
FocusNode get contentFocusNode => _contentFocusNode;
11071107
final _contentFocusNode = FocusNode();
11081108

11091109
@mustCallSuper
11101110
void dispose() {
1111-
_contentController.dispose();
1111+
_content.dispose();
11121112
_contentFocusNode.dispose();
11131113
}
11141114
}
11151115

11161116
class StreamComposeBoxController extends ComposeBoxController {
1117-
ComposeTopicController get topicController => _topicController;
1118-
final _topicController = ComposeTopicController();
1117+
ComposeTopicController get topic => _topic;
1118+
final _topic = ComposeTopicController();
11191119

11201120
FocusNode get topicFocusNode => _topicFocusNode;
11211121
final _topicFocusNode = FocusNode();
11221122

11231123
@override
11241124
void dispose() {
1125-
_topicController.dispose();
1125+
_topic.dispose();
11261126
_topicFocusNode.dispose();
11271127
super.dispose();
11281128
}
@@ -1157,29 +1157,29 @@ class _StreamComposeBoxState extends State<_StreamComposeBox> {
11571157
@override
11581158
Widget build(BuildContext context) {
11591159
final StreamComposeBoxController(
1160-
:topicController, :contentController, :topicFocusNode, :contentFocusNode,
1160+
:topic, :content, :topicFocusNode, :contentFocusNode,
11611161
) = controller;
11621162

11631163
return _ComposeBoxLayout(
1164-
contentController: contentController,
1164+
contentController: content,
11651165
contentFocusNode: contentFocusNode,
11661166
topicInput: _TopicInput(
11671167
streamId: widget.narrow.streamId,
1168-
controller: topicController,
1168+
controller: topic,
11691169
focusNode: topicFocusNode,
11701170
contentFocusNode: contentFocusNode,
11711171
),
11721172
contentInput: _StreamContentInput(
11731173
narrow: widget.narrow,
1174-
topicController: topicController,
1175-
controller: contentController,
1174+
topicController: topic,
1175+
controller: content,
11761176
focusNode: contentFocusNode,
11771177
),
11781178
sendButton: _SendButton(
1179-
topicController: topicController,
1180-
contentController: contentController,
1179+
topicController: topic,
1180+
contentController: content,
11811181
getDestination: () => StreamDestination(
1182-
widget.narrow.streamId, topicController.textNormalized),
1182+
widget.narrow.streamId, topic.textNormalized),
11831183
));
11841184
}
11851185
}
@@ -1227,21 +1227,21 @@ class _FixedDestinationComposeBoxState extends State<_FixedDestinationComposeBox
12271227
@override
12281228
Widget build(BuildContext context) {
12291229
final FixedDestinationComposeBoxController(
1230-
:contentController, :contentFocusNode,
1230+
:content, :contentFocusNode,
12311231
) = controller;
12321232

12331233
return _ComposeBoxLayout(
1234-
contentController: contentController,
1234+
contentController: content,
12351235
contentFocusNode: contentFocusNode,
12361236
topicInput: null,
12371237
contentInput: _FixedDestinationContentInput(
12381238
narrow: widget.narrow,
1239-
controller: contentController,
1239+
controller: content,
12401240
focusNode: contentFocusNode,
12411241
),
12421242
sendButton: _SendButton(
12431243
topicController: null,
1244-
contentController: contentController,
1244+
contentController: content,
12451245
getDestination: () => widget.narrow.destination,
12461246
));
12471247
}

‎test/widgets/action_sheet_test.dart

+5-5
Original file line numberDiff line numberDiff line change
@@ -284,12 +284,12 @@ void main() {
284284
await setupToMessageActionSheet(tester, message: message, narrow: ChannelNarrow(message.streamId));
285285

286286
final composeBoxController = findComposeBoxController(tester) as StreamComposeBoxController;
287-
final contentController = composeBoxController.contentController;
287+
final contentController = composeBoxController.content;
288288

289289
// Ensure channel-topics are loaded before testing quote & reply behavior
290290
connection.prepare(body:
291291
jsonEncode(GetStreamTopicsResult(topics: [eg.getStreamTopicsEntry()]).toJson()));
292-
final topicController = composeBoxController.topicController;
292+
final topicController = composeBoxController.topic;
293293
topicController.value = const TextEditingValue(text: kNoTopicTopic);
294294

295295
final valueBefore = contentController.value;
@@ -308,7 +308,7 @@ void main() {
308308
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
309309

310310
final composeBoxController = findComposeBoxController(tester)!;
311-
final contentController = composeBoxController.contentController;
311+
final contentController = composeBoxController.content;
312312

313313
final valueBefore = contentController.value;
314314
prepareRawContentResponseSuccess(message: message, rawContent: 'Hello world');
@@ -344,7 +344,7 @@ void main() {
344344
message: message, narrow: DmNarrow.ofMessage(message, selfUserId: eg.selfUser.userId));
345345

346346
final composeBoxController = findComposeBoxController(tester)!;
347-
final contentController = composeBoxController.contentController;
347+
final contentController = composeBoxController.content;
348348

349349
final valueBefore = contentController.value;
350350
prepareRawContentResponseSuccess(message: message, rawContent: 'Hello world');
@@ -383,7 +383,7 @@ void main() {
383383
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
384384

385385
final composeBoxController = findComposeBoxController(tester)!;
386-
final contentController = composeBoxController.contentController;
386+
final contentController = composeBoxController.content;
387387

388388
final valueBefore = contentController.value = TextEditingValue.empty;
389389
prepareRawContentResponseError();

‎test/widgets/compose_box_test.dart

+9-9
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ void main() {
204204
check(controller).isA<StreamComposeBoxController>();
205205
final topicTextField = tester.widgetList<TextField>(
206206
find.byWidgetPredicate((widget) {
207-
final topicController = (controller as StreamComposeBoxController).topicController;
207+
final topicController = (controller as StreamComposeBoxController).topic;
208208
return widget is TextField && widget.controller == topicController;
209209
})).singleOrNull;
210210
check(topicTextField).isNotNull()
@@ -216,7 +216,7 @@ void main() {
216216

217217
final contentTextField = tester.widget<TextField>(find.byWidgetPredicate(
218218
(widget) => widget is TextField
219-
&& widget.controller == controller.contentController));
219+
&& widget.controller == controller.content));
220220
check(contentTextField)
221221
.textCapitalization.equals(TextCapitalization.sentences);
222222
}
@@ -364,7 +364,7 @@ void main() {
364364
checkTypingRequest(TypingOp.stop, narrow);
365365

366366
connection.prepare(json: {});
367-
controller.contentController.selection =
367+
controller.content.selection =
368368
const TextSelection(baseOffset: 0, extentOffset: 2);
369369
checkTypingRequest(TypingOp.start, narrow);
370370

@@ -480,7 +480,7 @@ void main() {
480480
// (When we check that the send button looks disabled, it should be because
481481
// the file is uploading, not a pre-existing reason.)
482482
await enterTopic(tester, narrow: narrow, topic: 'some topic');
483-
controller.contentController.value = const TextEditingValue(text: 'see image: ');
483+
controller.content.value = const TextEditingValue(text: 'see image: ');
484484
await tester.pump();
485485
checkAppearsLoading(tester, false);
486486

@@ -504,7 +504,7 @@ void main() {
504504
final errorDialogs = tester.widgetList(find.byType(AlertDialog));
505505
check(errorDialogs).isEmpty();
506506

507-
check(controller.contentController.text)
507+
check(controller.content.text)
508508
.equals('see image: [Uploading image.jpg…]()\n\n');
509509
// (the request is checked more thoroughly in API tests)
510510
check(connection.lastRequest!).isA<http.MultipartRequest>()
@@ -520,7 +520,7 @@ void main() {
520520
checkAppearsLoading(tester, true);
521521

522522
await tester.pump(const Duration(seconds: 1));
523-
check(controller.contentController.text)
523+
check(controller.content.text)
524524
.equals('see image: [image.jpg](/user_uploads/1/4e/m2A3MSqFnWRLUf9SaPzQ0Up_/image.jpg)\n\n');
525525
checkAppearsLoading(tester, false);
526526
});
@@ -541,7 +541,7 @@ void main() {
541541
// (When we check that the send button looks disabled, it should be because
542542
// the file is uploading, not a pre-existing reason.)
543543
await enterTopic(tester, narrow: narrow, topic: 'some topic');
544-
controller.contentController.value = const TextEditingValue(text: 'see image: ');
544+
controller.content.value = const TextEditingValue(text: 'see image: ');
545545
await tester.pump();
546546
checkAppearsLoading(tester, false);
547547

@@ -565,7 +565,7 @@ void main() {
565565
final errorDialogs = tester.widgetList(find.byType(AlertDialog));
566566
check(errorDialogs).isEmpty();
567567

568-
check(controller.contentController.text)
568+
check(controller.content.text)
569569
.equals('see image: [Uploading image.jpg…]()\n\n');
570570
// (the request is checked more thoroughly in API tests)
571571
check(connection.lastRequest!).isA<http.MultipartRequest>()
@@ -581,7 +581,7 @@ void main() {
581581
checkAppearsLoading(tester, true);
582582

583583
await tester.pump(const Duration(seconds: 1));
584-
check(controller.contentController.text)
584+
check(controller.content.text)
585585
.equals('see image: [image.jpg](/user_uploads/1/4e/m2A3MSqFnWRLUf9SaPzQ0Up_/image.jpg)\n\n');
586586
checkAppearsLoading(tester, false);
587587
});

0 commit comments

Comments
 (0)