fixed list view breaking layout

Reviewed-on: #93
Reviewed-by: Elias Müller <elias@elias-mueller.com>

devtools_options.yaml

@@ -1 +1,4 @@
 extensions:
+  - hive_ce: true
+  - shared_preferences: true
+  - provider: true

lib/api/marianumcloud/talk/getPoll/getPollState.dart

@@ -0,0 +1,18 @@
+import 'dart:convert';
+
+import 'package:http/http.dart' as http;
+
+import '../talkApi.dart';
+import 'getPollStateResponse.dart';
+
+class GetPollState extends TalkApi<GetPollStateResponse> {
+  String token;
+  int pollId;
+  GetPollState({required this.token, required this.pollId}) : super('v1/poll/$token/$pollId', null);
+
+  @override
+  GetPollStateResponse assemble(String raw) => GetPollStateResponse.fromJson(jsonDecode(raw)['ocs']);
+
+  @override
+  Future<http.Response> request(Uri uri, Object? body, Map<String, String>? headers) => http.get(uri, headers: headers);
+}

lib/api/marianumcloud/talk/getPoll/getPollStateResponse.dart

@@ -0,0 +1,50 @@
+import 'package:json_annotation/json_annotation.dart';
+
+import '../../../apiResponse.dart';
+
+part 'getPollStateResponse.g.dart';
+
+@JsonSerializable(explicitToJson: true)
+class GetPollStateResponse extends ApiResponse {
+  GetPollStateResponseObject data;
+
+  GetPollStateResponse(this.data);
+
+  factory GetPollStateResponse.fromJson(Map<String, dynamic> json) => _$GetPollStateResponseFromJson(json);
+  Map<String, dynamic> toJson() => _$GetPollStateResponseToJson(this);
+}
+
+@JsonSerializable(explicitToJson: true)
+class GetPollStateResponseObject {
+  int id;
+  String question;
+  List<String> options;
+  dynamic votes;
+  String actorType;
+  String actorId;
+  String actorDisplayName;
+  int status;
+  int resultMode;
+  int maxVotes;
+  List<int> votedSelf;
+  int? numVoters;
+  List<dynamic>? details;
+
+  GetPollStateResponseObject(
+      this.id,
+      this.question,
+      this.options,
+      this.votes,
+      this.actorType,
+      this.actorId,
+      this.actorDisplayName,
+      this.status,
+      this.resultMode,
+      this.maxVotes,
+      this.votedSelf,
+      this.numVoters,
+      this.details);
+
+  factory GetPollStateResponseObject.fromJson(Map<String, dynamic> json) => _$GetPollStateResponseObjectFromJson(json);
+  Map<String, dynamic> toJson() => _$GetPollStateResponseObjectToJson(this);
+}

lib/api/marianumcloud/talk/getPoll/getPollStateResponse.g.dart

@@ -0,0 +1,62 @@
+// GENERATED CODE - DO NOT MODIFY BY HAND
+
+part of 'getPollStateResponse.dart';
+
+// **************************************************************************
+// JsonSerializableGenerator
+// **************************************************************************
+
+GetPollStateResponse _$GetPollStateResponseFromJson(
+  Map<String, dynamic> json,
+) =>
+    GetPollStateResponse(
+        GetPollStateResponseObject.fromJson(
+          json['data'] as Map<String, dynamic>,
+        ),
+      )
+      ..headers = (json['headers'] as Map<String, dynamic>?)?.map(
+        (k, e) => MapEntry(k, e as String),
+      );
+
+Map<String, dynamic> _$GetPollStateResponseToJson(
+  GetPollStateResponse instance,
+) => <String, dynamic>{
+  'headers': ?instance.headers,
+  'data': instance.data.toJson(),
+};
+
+GetPollStateResponseObject _$GetPollStateResponseObjectFromJson(
+  Map<String, dynamic> json,
+) => GetPollStateResponseObject(
+  (json['id'] as num).toInt(),
+  json['question'] as String,
+  (json['options'] as List<dynamic>).map((e) => e as String).toList(),
+  json['votes'],
+  json['actorType'] as String,
+  json['actorId'] as String,
+  json['actorDisplayName'] as String,
+  (json['status'] as num).toInt(),
+  (json['resultMode'] as num).toInt(),
+  (json['maxVotes'] as num).toInt(),
+  (json['votedSelf'] as List<dynamic>).map((e) => (e as num).toInt()).toList(),
+  (json['numVoters'] as num?)?.toInt(),
+  json['details'] as List<dynamic>?,
+);
+
+Map<String, dynamic> _$GetPollStateResponseObjectToJson(
+  GetPollStateResponseObject instance,
+) => <String, dynamic>{
+  'id': instance.id,
+  'question': instance.question,
+  'options': instance.options,
+  'votes': instance.votes,
+  'actorType': instance.actorType,
+  'actorId': instance.actorId,
+  'actorDisplayName': instance.actorDisplayName,
+  'status': instance.status,
+  'resultMode': instance.resultMode,
+  'maxVotes': instance.maxVotes,
+  'votedSelf': instance.votedSelf,
+  'numVoters': instance.numVoters,
+  'details': instance.details,
+};

lib/api/marianumcloud/talk/votePoll/votePoll.dart

@@ -0,0 +1,26 @@
+import 'dart:convert';
+
+import 'package:http/http.dart' as http;
+import 'package:http/http.dart';
+
+import '../getPoll/getPollStateResponse.dart';
+import '../talkApi.dart';
+import 'votePollParams.dart';
+
+@Deprecated('VotePoll is broken')
+class VotePoll extends TalkApi {
+  String token;
+  int pollId;
+  VotePoll({required this.token, required this.pollId, required VotePollParams params}) : super('v1/poll/$token/$pollId', params);
+
+  @override
+  GetPollStateResponse assemble(String raw) => GetPollStateResponse.fromJson(jsonDecode(raw)['ocs']);
+
+  @override
+  Future<Response>? request(Uri uri, Object? body, Map<String, String>? headers) {
+    if(body is VotePollParams) {
+      return http.post(uri, headers: headers, body: body.toJson().toString());
+    }
+    return null;
+  }
+}

lib/api/marianumcloud/talk/votePoll/votePollParams.dart

@@ -0,0 +1,15 @@
+import 'package:json_annotation/json_annotation.dart';
+
+import '../../../apiParams.dart';
+
+part 'votePollParams.g.dart';
+
+@JsonSerializable()
+@Deprecated('VotePoll is broken')
+class VotePollParams extends ApiParams {
+  List<int> optionIds;
+
+  VotePollParams({required this.optionIds});
+  factory VotePollParams.fromJson(Map<String, dynamic> json) => _$VotePollParamsFromJson(json);
+  Map<String, dynamic> toJson() => _$VotePollParamsToJson(this);
+}

lib/api/marianumcloud/talk/votePoll/votePollParams.g.dart

@@ -0,0 +1,17 @@
+// GENERATED CODE - DO NOT MODIFY BY HAND
+
+part of 'votePollParams.dart';
+
+// **************************************************************************
+// JsonSerializableGenerator
+// **************************************************************************
+
+VotePollParams _$VotePollParamsFromJson(Map<String, dynamic> json) =>
+    VotePollParams(
+      optionIds: (json['optionIds'] as List<dynamic>)
+          .map((e) => (e as num).toInt())
+          .toList(),
+    );
+
+Map<String, dynamic> _$VotePollParamsToJson(VotePollParams instance) =>
+    <String, dynamic>{'optionIds': instance.optionIds};

lib/state/app/modules/gradeAverages/view/grade_averages_view.dart

@@ -51,7 +51,7 @@ class GradeAveragesView extends StatelessWidget {
                           color: Theme.of(context).colorScheme.onSurface
                       ),
                       const SizedBox(width: 15),
-                      Text(isMiddleSchool ? 'Notensystem' : 'Punktesystem'),
+                      Text(isMiddleSchool ? 'Realschule' : 'Oberstufe'),
                     ],
                   ),
                 )).toList(),
@@ -80,11 +80,19 @@ class GradeAveragesView extends StatelessWidget {
             mainAxisAlignment: MainAxisAlignment.center,
             children: [
               const SizedBox(height: 30),
-              Text(bloc.average().toStringAsFixed(2), style: const TextStyle(fontSize: 30, fontWeight: FontWeight.bold)),
+              Row(
+                crossAxisAlignment: CrossAxisAlignment.center,
+                mainAxisAlignment: MainAxisAlignment.center,
+                children: [
+                  Text('Ø', style: TextStyle(fontSize: 30, fontWeight: FontWeight.bold)),
+                  SizedBox(width: 5),
+                  Text(bloc.average().toStringAsFixed(2), style: const TextStyle(fontSize: 30, fontWeight: FontWeight.bold))
+                ],
+              ),
               const SizedBox(height: 10),
               const Divider(),
               const SizedBox(height: 10),
-              Text(bloc.isMiddleSchool() ? 'Wähle unten die Anzahl deiner jeweiligen Noten aus' : 'Wähle unten die Anzahl deiner jeweiligen Punkte aus'),
+              Text(bloc.isMiddleSchool() ? 'Wähle die Anzahl deiner jeweiligen Noten aus' : 'Wähle die Anzahl deiner jeweiligen Punkte aus'),
               const SizedBox(height: 10),
               const Expanded(
                 child: GradeAveragesListView()

lib/utils/FileSaver.dart

@@ -0,0 +1,23 @@
+import 'dart:io';
+
+import 'package:permission_handler/permission_handler.dart';
+
+// only tested on android!
+class FileSaver {
+  static Future<String> getExternalDocumentPath() async {
+    var permission = await Permission.storage.status;
+    if(!permission.isGranted) {
+      await Permission.storage.request();
+    }
+    var directory = Directory('/storage/emulated/0/Download');
+    final externalPath = directory.path;
+    await Directory(externalPath).create(recursive: true);
+    return externalPath;
+  }
+
+  static Future<File> writeBytes(List<int> bytes, String name) async {
+    final path = await getExternalDocumentPath();
+    var file = File('$path/$name');
+    return file.writeAsBytes(bytes);
+  }
+}

lib/utils/UrlOpener.dart

@@ -0,0 +1,10 @@
+import 'package:flutter_linkify/flutter_linkify.dart';
+import 'package:url_launcher/url_launcher_string.dart';
+
+class UrlOpener {
+  static Future<void> onOpen(LinkableElement link) async {
+    if(await canLaunchUrlString(link.url)) {
+      await launchUrlString(link.url);
+    }
+  }
+}

lib/view/pages/more/share/qrShareView.dart

@@ -1,4 +1,5 @@
 import 'package:flutter/material.dart';
+import 'package:screen_brightness/screen_brightness.dart';
 
 import 'appSharePlatformView.dart';
 
@@ -10,6 +11,18 @@ class QrShareView extends StatefulWidget {
 }
 
 class _QrShareViewState extends State<QrShareView> {
+  @override
+  void initState() {
+    ScreenBrightness.instance.setApplicationScreenBrightness(1.0);
+    super.initState();
+  }
+
+  @override
+  void dispose() {
+    ScreenBrightness.instance.resetApplicationScreenBrightness();
+    super.dispose();
+  }
+
   @override
   Widget build(BuildContext context) => DefaultTabController(
       length: 2,

lib/view/pages/talk/chatView.dart

@@ -51,6 +51,7 @@ class _ChatViewState extends State<ChatView> {
             var elementDate = DateTime.fromMillisecondsSinceEpoch(element.timestamp * 1000);
 
             if(element.systemMessage.contains('reaction')) return;
+            if(element.systemMessage.contains('poll_voted')) return;
             var commonRead = int.parse(data.getChatResponse.headers?['x-chat-last-common-read'] ?? '0');
 
             if(!elementDate.isSameDay(lastDate)) {
@@ -128,7 +129,7 @@ class _ChatViewState extends State<ChatView> {
                   ),
                 ),
                 Container(
-                  color: Theme.of(context).colorScheme.background,
+                  color: Theme.of(context).colorScheme.surface,
                   child: TalkNavigator.isSecondaryVisible(context)
                       ? ChatTextfield(widget.room.token, selfId: widget.selfId)
                       : SafeArea(child: ChatTextfield(widget.room.token, selfId: widget.selfId)

lib/view/pages/talk/components/chatBubble.dart

@@ -6,6 +6,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:jiffy/jiffy.dart';
 import 'package:open_filex/open_filex.dart';
+import '../../../../api/marianumcloud/talk/getPoll/getPollState.dart';
 import '../../../../extensions/text.dart';
 import 'package:provider/provider.dart';
 
@@ -18,11 +19,13 @@ import '../../../../api/marianumcloud/talk/reactMessage/reactMessageParams.dart'
 import '../../../../api/marianumcloud/talk/room/getRoomResponse.dart';
 import '../../../../model/chatList/chatProps.dart';
 import '../../../../widget/debug/debugTile.dart';
+import '../../../../widget/loadingSpinner.dart';
 import '../../files/fileElement.dart';
 import 'answerReference.dart';
 import 'chatBubbleStyles.dart';
 import 'chatMessage.dart';
 import '../messageReactions.dart';
+import 'pollOptionsList.dart';
 
 class ChatBubble extends StatefulWidget {
   final BuildContext context;
@@ -297,6 +300,34 @@ class _ChatBubbleState extends State<ChatBubble> with SingleTickerProviderStateM
           onLongPress: showOptionsDialog,
           onDoubleTap: showOptionsDialog,
           onTap: () {
+            if(message.originalData?['object']?.type == RichObjectStringObjectType.talkPoll) {
+              var pollId = int.parse(message.originalData!['object']!.id);
+              var pollState = GetPollState(token: widget.bubbleData.token, pollId: pollId).run();
+              showDialog(context: context, builder: (context) => AlertDialog(
+                title: Text(message.originalData!['object']!.name, overflow: TextOverflow.ellipsis),
+                content: FutureBuilder(
+                  future: pollState,
+                  builder: (context, snapshot) {
+                    if(snapshot.connectionState == ConnectionState.waiting) return const Column(mainAxisSize: MainAxisSize.min, children: [LoadingSpinner()]);
+
+                    var pollData = snapshot.data!.data;
+                    return SingleChildScrollView(
+                      child: PollOptionsList(
+                        pollData: pollData,
+                        chatToken: widget.chatData.token,
+                      ),
+                    );
+                  }
+                ),
+                actions: [
+                  TextButton(
+                    onPressed: () => Navigator.of(context).pop(),
+                    child: const Text('Zurück')
+                  ),
+                ],
+              ));
+            }
+
             if(message.file == null) return;
 
             if(downloadProgress > 0) {

lib/view/pages/talk/components/chatMessage.dart

@@ -2,12 +2,12 @@
 import 'package:cached_network_image/cached_network_image.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_linkify/flutter_linkify.dart';
-import 'package:url_launcher/url_launcher_string.dart';
 
 import '../../../../api/marianumcloud/talk/chat/getChatResponse.dart';
 import '../../../../api/marianumcloud/talk/chat/richObjectStringProcessor.dart';
 import '../../../../model/accountData.dart';
 import '../../../../model/endpointData.dart';
+import '../../../../utils/UrlOpener.dart';
 
 class ChatMessage {
   String originalMessage;
@@ -29,9 +29,17 @@ class ChatMessage {
 
     var contentWidget = Linkify(
       text: content,
-      onOpen: onOpen,
+      onOpen: UrlOpener.onOpen,
     );
 
+    if(originalData?['object']?.type == RichObjectStringObjectType.talkPoll) {
+      return ListTile(
+        leading: const Icon(Icons.poll_outlined),
+        title: Text(originalData!['object']!.name),
+        contentPadding: const EdgeInsets.only(left: 10),
+      );
+    }
+
     if(file == null) return contentWidget;
 
     return Padding(
@@ -65,10 +73,4 @@ class ChatMessage {
         )
     );
   }
-
-  Future<void> onOpen(LinkableElement link) async {
-    if(await canLaunchUrlString(link.url)) {
-      await launchUrlString(link.url);
-    }
-  }
 }

lib/view/pages/talk/components/pollOptionsList.dart

@@ -0,0 +1,67 @@
+
+import 'package:flutter/material.dart';
+import 'package:flutter_linkify/flutter_linkify.dart';
+
+import '../../../../api/marianumcloud/talk/getPoll/getPollStateResponse.dart';
+import '../../../../utils/UrlOpener.dart';
+
+class PollOptionsList extends StatefulWidget {
+  final GetPollStateResponseObject pollData;
+  final String chatToken;
+  const PollOptionsList({super.key, required this.pollData, required this.chatToken});
+
+  @override
+  State<PollOptionsList> createState() => _PollOptionsListState();
+}
+
+class _PollOptionsListState extends State<PollOptionsList> {
+  @override
+  Widget build(BuildContext context) => Column(
+    children: [
+      ...widget.pollData.options.map<Widget>((option) {
+        var optionId = widget.pollData.options.indexOf(option);
+        var votedSelf = widget.pollData.votedSelf.contains(optionId);
+        var portionsVisible = widget.pollData.votes is Map<String, dynamic>;
+        var votes = portionsVisible
+          ? (widget.pollData.votes['option-$optionId'] as num?) ?? 0
+          : 0;
+        var numVoters = widget.pollData.numVoters ?? 0;
+        double portion = numVoters == 0 ? 0 : (votes / numVoters);
+
+        return ListTile(
+          // enabled: false,
+          isThreeLine: portionsVisible,
+          dense: true,
+          title: Text(
+            option,
+            style: Theme.of(context).textTheme.bodyLarge,
+          ),
+          leading: Icon(
+            votedSelf ? Icons.check_circle_outlined : Icons.circle_outlined,
+            color: votedSelf
+                ? Theme.of(context).colorScheme.primary.withValues(alpha: 0.6)
+                : Theme.of(context).colorScheme.onSurfaceVariant.withValues(alpha: 0.6),
+          ),
+          subtitle: portionsVisible ? Row(
+              children: [
+                Expanded(
+                  child: LinearProgressIndicator(value: portion.clamp(0.0, 1.0)),
+                ),
+                Container(
+                  margin: const EdgeInsets.only(left: 10),
+                  child: Text('${(portion * 100).round()}%'),
+                ),
+              ],
+            ) : null,
+        );
+      }),
+      ListTile(
+        title: Linkify(
+          text: 'Wenn du abstimmen möchtest, verwende die Webversion unter https://cloud.marianum-fulda.de/call/${widget.chatToken}',
+          onOpen: UrlOpener.onOpen,
+          style: Theme.of(context).textTheme.bodySmall,
+        ),
+      )
+    ],
+  );
+}

lib/widget/fileViewer.dart

@@ -9,6 +9,8 @@ import 'package:share_plus/share_plus.dart';
 import 'package:syncfusion_flutter_pdfviewer/pdfviewer.dart';
 
 import '../storage/base/settingsProvider.dart';
+import '../utils/FileSaver.dart';
+import 'infoDialog.dart';
 import 'placeholderView.dart';
 import 'sharePositionOrigin.dart';
 
@@ -21,6 +23,12 @@ class FileViewer extends StatefulWidget {
   State<FileViewer> createState() => _FileViewerState();
 }
 
+enum FileViewingActions {
+  openExternal,
+  share,
+  save
+}
+
 class _FileViewerState extends State<FileViewer> {
   PhotoViewController photoViewController = PhotoViewController();
 
@@ -38,22 +46,57 @@ class _FileViewerState extends State<FileViewer> {
     AppBar appbar({List actions = const []}) => AppBar(
         title: Text(widget.path.split('/').last),
         actions: [
-          IconButton(
+          ...actions,
-              onPressed: () => Navigator.of(context).push(
+          PopupMenuButton<FileViewingActions>(
+            onSelected: (value) async {
+              switch(value) {
+                case FileViewingActions.openExternal:
+                  Navigator.of(context).push(
                       MaterialPageRoute(builder: (context) => FileViewer(path: widget.path, openExternal: true))
-              ),
-              icon: const Icon(Icons.open_in_new)
-          ),
-          IconButton(
-            onPressed: () {
-              Share.shareXFiles(
-                  [XFile(widget.path)],
-                  sharePositionOrigin: SharePositionOrigin.get(context),
                   );
+                  break;
+                case FileViewingActions.share:
+                  SharePlus.instance.share(
+                      ShareParams(
+                          files: [XFile(widget.path)],
+                          sharePositionOrigin: SharePositionOrigin.get(context)
+                      )
+                  );
+                  break;
+                case FileViewingActions.save:
+                  await FileSaver.writeBytes(await File(widget.path).readAsBytes(), widget.path.split('/').last);
+                  if(!context.mounted) return;
+                  InfoDialog.show(context, 'Die Datei wurde im Downloads Ordner gespeichert.');
+                  break;
+              }
             },
-            icon: const Icon(Icons.share_outlined),
+            itemBuilder: (context) => <PopupMenuEntry<FileViewingActions>>[
+              const PopupMenuItem(
+                value: FileViewingActions.openExternal,
+                child: ListTile(
+                  leading: Icon(Icons.open_in_new),
+                  title: Text('Extern öffnen'),
+                  dense: true,
+                ),
+              ),
+              const PopupMenuItem(
+                value: FileViewingActions.share,
+                child: ListTile(
+                  leading: Icon(Icons.share_outlined),
+                  title: Text('Teilen'),
+                  dense: true,
+                ),
+              ),
+              if(Platform.isAndroid) const PopupMenuItem(
+                value: FileViewingActions.save,
+                child: ListTile(
+                  leading: Icon(Icons.save_alt_outlined),
+                  title: Text('Speichern'),
+                  dense: true,
+                ),
+              ),
+            ],
           ),
-          ...actions
         ],
       );
 

pubspec.yaml

@@ -64,6 +64,7 @@ dependencies:
   open_filex: ^4.7.0
   package_info_plus: ^8.1.3
   path_provider: ^2.1.5
+  permission_handler: ^12.0.1
   persistent_bottom_nav_bar_v2: ^6.1.0
   photo_view: ^0.15.0
   pretty_json: ^2.0.0
@@ -71,6 +72,7 @@ dependencies:
   qr_flutter: ^4.1.0
   rrule: ^0.2.17
   rrule_generator: ^0.9.0
+  screen_brightness: ^2.1.7
   share_plus: ^11.1.0
   shared_preferences: ^2.3.5
   syncfusion_flutter_calendar: ^31.1.17