ember/fluffychat
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

1594-activity-planner-feedback-new (#1599)

Browse source 
* feat(activity_planner): updating for feedback

* fix(main): point to appropriate env

---------

Co-authored-by: ggurdin <ggurdin@gmail.com>
This commit is contained in:
wcjord 2025-01-27 14:13:50 -05:00 • committed by GitHub
parent e2b991b36b
commit 8478f480d3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
7 changed files with 138 additions and 52 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
assets/l10n/intl_en.arb
View file

@ -4757,7 +4757,7 @@
"modePlaceholder": "Choose a mode...",
"learningObjectiveLabel": "Learning Objective",
"learningObjectivePlaceholder": "Choose a learning objective...",
"mediaLabel": "Media students should share",
"mediaLabel": "Media learners should share",
"languageOfInstructionsLabel": "Language of activity instructions",
"targetLanguageLabel": "Target language",
"cefrLevelLabel": "CEFR Level",
@ -4770,6 +4770,13 @@
"activityPlannerOverviewInstructionsBody": "Choose a topic, mode, learning objective and generate an activity for the chat!",
"completeActivitiesToUnlock": "Complete the highlighted word activities to unlock",
"myBookmarkedActivities": "My Bookmarked Activities",
"noBookmarkedActivities": "When you bookmark activities, they will appear here. Bookmarked activities can be re-used across spaces and chats.",
"activityTitle": "Activity Title",
"addVocabulary": "Add Vocabulary",
"instructions": "Instructions",
"bookmark": "Bookmark this activity",
"numberOfLearners": "Number of learners",
"mustBeInteger": "Must be an integer e.g. 1, 2, 3, ...",
"noBookmarkedActivities": "No bookmarked activities",
"noLemmasFound": "No lemmas found"
}

10
lib/pangea/activity_planner/activity_list_view.dart
View file

@ -124,7 +124,15 @@ class ActivityListViewState extends State<ActivityListView> {
);
} else if (!snapshot.hasData || snapshot.data!.isEmpty) {
if (showBookmarkedActivities) {
return Center(child: Text(l10n.noBookmarkedActivities));
return Center(
child: Container(
constraints: const BoxConstraints(maxWidth: 200),
child: Text(
l10n.noBookmarkedActivities,
textAlign: TextAlign.center,
),
),
);
}
return Center(child: Text(l10n.noDataFound));
} else {

99
lib/pangea/activity_planner/activity_plan_card.dart
View file

@ -35,6 +35,7 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
late TextEditingController _learningObjectiveController;
late TextEditingController _instructionsController;
final TextEditingController _newVocabController = TextEditingController();
final FocusNode _vocabFocusNode = FocusNode();
@override
void initState() {
@ -47,7 +48,7 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
TextEditingController(text: _tempActivity.instructions);
}
static const double itemPadding = 8;
static const double itemPadding = 12;
@override
void dispose() {
@ -55,6 +56,7 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
_learningObjectiveController.dispose();
_instructionsController.dispose();
_newVocabController.dispose();
_vocabFocusNode.dispose();
super.dispose();
}
@ -99,6 +101,7 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
setState(() {
_tempActivity.vocab.add(Vocab(lemma: _newVocabController.text, pos: ''));
_newVocabController.clear();
_vocabFocusNode.requestFocus();
});
}
@ -134,7 +137,7 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
? TextField(
controller: _titleController,
decoration: InputDecoration(
labelText: L10n.of(context).title,
labelText: L10n.of(context).activityTitle,
),
maxLines: null,
)
@ -166,8 +169,8 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
child: _isEditing
? TextField(
controller: _learningObjectiveController,
decoration: const InputDecoration(
labelText: 'Learning Objective',
decoration: InputDecoration(
labelText: l10n.learningObjectiveLabel,
),
maxLines: null,
)
@ -190,8 +193,8 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
child: _isEditing
? TextField(
controller: _instructionsController,
decoration: const InputDecoration(
labelText: 'Instructions',
decoration: InputDecoration(
labelText: l10n.instructions,
),
maxLines: null,
)
@ -244,50 +247,62 @@ class ActivityPlanCardState extends State<ActivityPlanCard> {
),
],
),
if (_isEditing)
Padding(
padding: const EdgeInsets.only(top: itemPadding),
child: Row(
children: [
Expanded(
child: TextField(
controller: _newVocabController,
decoration: const InputDecoration(
labelText: 'Add Vocabulary',
),
),
),
IconButton(
icon: const Icon(Icons.add),
onPressed: _addVocab,
),
],
),
),
],
const SizedBox(height: 16),
if (_isEditing) ...[
const SizedBox(height: itemPadding),
Padding(
padding: const EdgeInsets.only(top: itemPadding),
child: Row(
children: [
Expanded(
child: TextField(
controller: _newVocabController,
focusNode: _vocabFocusNode,
decoration: InputDecoration(
labelText: l10n.addVocabulary,
),
onSubmitted: (value) {
_addVocab();
},
),
),
IconButton(
icon: const Icon(Icons.add),
onPressed: _addVocab,
),
],
),
),
],
const SizedBox(height: itemPadding),
Row(
mainAxisAlignment: MainAxisAlignment.spaceBetween,
children: [
Row(
children: [
IconButton(
icon: Icon(!_isEditing ? Icons.edit : Icons.save),
onPressed: () => !_isEditing
? setState(() {
_isEditing = true;
})
: _saveEdits(),
isSelected: _isEditing,
Tooltip(
message: !_isEditing ? l10n.edit : l10n.saveChanges,
child: IconButton(
icon: Icon(!_isEditing ? Icons.edit : Icons.save),
onPressed: () => !_isEditing
? setState(() {
_isEditing = true;
})
: _saveEdits(),
isSelected: _isEditing,
),
),
if (_isEditing)
IconButton(
icon: const Icon(Icons.cancel),
onPressed: () {
setState(() {
_isEditing = false;
});
},
Tooltip(
message: l10n.cancel,
child: IconButton(
icon: const Icon(Icons.cancel),
onPressed: () {
setState(() {
_isEditing = false;
});
},
),
),
],
),

6
lib/pangea/activity_planner/activity_plan_request.dart
View file

@ -9,6 +9,7 @@ class ActivityPlanRequest {
final String languageOfInstructions;
final String targetLanguage;
final int count;
final int numberOfParticipants;
ActivityPlanRequest({
required this.topic,
@ -19,6 +20,7 @@ class ActivityPlanRequest {
required this.languageOfInstructions,
required this.targetLanguage,
this.count = 3,
required this.numberOfParticipants,
});
Map<String, dynamic> toJson() {
@ -31,6 +33,7 @@ class ActivityPlanRequest {
'language_of_instructions': languageOfInstructions,
'target_language': targetLanguage,
'count': count,
'number_of_participants': numberOfParticipants,
};
}
@ -68,11 +71,12 @@ class ActivityPlanRequest {
languageOfInstructions: json['language_of_instructions'],
targetLanguage: json['target_language'],
count: json['count'],
numberOfParticipants: json['number_of_participants'],
);
}
String get storageKey =>
'$topic-$mode-$objective-${media.string}-$cefrLevel-$languageOfInstructions-$targetLanguage';
'$topic-$mode-$objective-${media.string}-$cefrLevel-$languageOfInstructions-$targetLanguage-$numberOfParticipants';
String get cefrLanguageLevel {
switch (cefrLevel) {

58
lib/pangea/activity_planner/activity_planner_page.dart
View file

@ -1,3 +1,5 @@
import 'dart:math';
import 'package:flutter/material.dart';
import 'package:dropdown_button2/dropdown_button2.dart';
@ -47,6 +49,7 @@ class ActivityPlannerPageState extends State<ActivityPlannerPage> {
String? _selectedLanguageOfInstructions;
String? _selectedTargetLanguage;
int? _selectedCefrLevel;
int? _selectedNumberOfParticipants;
List<String> activities = [];
@ -65,6 +68,7 @@ class ActivityPlannerPageState extends State<ActivityPlannerPage> {
_selectedTargetLanguage =
MatrixState.pangeaController.languageController.userL2?.langCode;
_selectedCefrLevel = 0;
_selectedNumberOfParticipants = max(room?.getParticipants().length ?? 1, 1);
}
final _topicController = TextEditingController();
@ -149,13 +153,34 @@ class ActivityPlannerPageState extends State<ActivityPlannerPage> {
icon: const Icon(Icons.arrow_back),
),
title: _pageMode == _PageMode.savedActivities
? Text(l10n.myBookmarkedActivities)
: Text(l10n.activityPlannerTitle),
? Center(
child: Row(
mainAxisSize: MainAxisSize.min,
children: [
const Icon(Icons.bookmarks),
const SizedBox(width: 8),
Text(l10n.myBookmarkedActivities),
],
),
)
: Center(
child: Row(
mainAxisSize: MainAxisSize.min,
children: [
const Icon(Icons.event_note_outlined),
const SizedBox(width: 8),
Text(l10n.activityPlannerTitle),
],
),
),
actions: [
IconButton(
onPressed: () =>
setState(() => _pageMode = _PageMode.savedActivities),
icon: const Icon(Icons.bookmarks),
Tooltip(
message: l10n.myBookmarkedActivities,
child: IconButton(
onPressed: () =>
setState(() => _pageMode = _PageMode.savedActivities),
icon: const Icon(Icons.bookmarks),
),
),
],
),
@ -172,6 +197,7 @@ class ActivityPlannerPageState extends State<ActivityPlannerPage> {
languageOfInstructions: _selectedLanguageOfInstructions!,
targetLanguage: _selectedTargetLanguage!,
cefrLevel: _selectedCefrLevel!,
numberOfParticipants: _selectedNumberOfParticipants!,
),
)
: Center(
@ -279,6 +305,26 @@ class ActivityPlannerPageState extends State<ActivityPlannerPage> {
isL2List: true,
),
const SizedBox(height: 24),
TextFormField(
decoration: InputDecoration(
labelText: l10n.numberOfLearners,
),
keyboardType: TextInputType.number,
validator: (value) {
if (value == null || value.isEmpty) {
return l10n.mustBeInteger;
}
final n = int.tryParse(value);
if (n == null || n <= 0) {
return l10n.mustBeInteger;
}
return null;
},
onChanged: (val) =>
_selectedNumberOfParticipants = int.tryParse(val),
initialValue: _selectedNumberOfParticipants?.toString(),
),
const SizedBox(height: 24),
ElevatedButton(
onPressed: () {
if (_formKey.currentState?.validate() ?? false) {

3
lib/pangea/activity_planner/suggestion_form_field.dart
View file

@ -43,6 +43,9 @@ class SuggestionFormField extends StatelessWidget {
VoidCallback onFieldSubmitted,
) {
textEditingController.value = controller.value;
textEditingController.addListener(() {
onSelected(textEditingController.text);
});
return TextFormField(
controller: textEditingController,
focusNode: focusNode,

5
lib/pangea/toolbar/widgets/overlay_header.dart
View file

@ -6,6 +6,7 @@ import 'package:matrix/matrix.dart';
import 'package:fluffychat/config/app_config.dart';
import 'package:fluffychat/pages/chat/chat.dart';
import 'package:fluffychat/pangea/common/constants/model_keys.dart';
class OverlayHeader extends StatelessWidget {
final ChatController controller;
@ -53,7 +54,9 @@ class OverlayHeader extends StatelessWidget {
tooltip: L10n.of(context).pinMessage,
color: Theme.of(context).colorScheme.primary,
),
if (controller.canEditSelectedEvents)
if (controller.canEditSelectedEvents &&
controller.selectedEvents.first.content[ModelKey.messageTags] !=
ModelKey.messageTagActivityPlan)
IconButton(
icon: const Icon(Icons.edit_outlined),
tooltip: L10n.of(context).edit,