Expandable room icons

Needs looking into once someone wants to help develop on Windows

.github/workflows/windows.yml

@@ -23,6 +23,13 @@ jobs:
 
       - name: Set up Go
         uses: actions/setup-go@v6
+        with:
+          go-version-file: gomuks/go.mod
+
+      - name: Go build
+        run: |
+          cd gomuks/pkg/ffi
+          go build -tags goolm -o ../../../libgomuks.dll -buildmode=c-shared
 
       - name: Build with Flutter
         run: |

hook/build.dart

@@ -18,7 +18,6 @@ Future<void> main(List<String> args) => build(args, (input, output) async {
       break;
     case OS.windows:
       libFileName = "libgomuks.dll";
-      env = {"GOCACHE": r"C:\Users\runneradmin\AppData\Local\go-build"};
       break;
     case OS.android:
       libFileName = "libgomuks.so";
@@ -56,9 +55,8 @@ Future<void> main(List<String> args) => build(args, (input, output) async {
 
     // goheif/dav1d supported on Android would need to fix upstream
     final tags = targetOS == OS.android ? "goolm,noheic" : "goolm";
-
     print(
-      "Building Gomuks shared library $libFileName (${targetOS.name}/${targetArch.name}) from source...",
+      "Building Gomuks shared library $libFileName (${targetOS.name}/${targetArch.name}) to ${libFile.path}...",
     );
     final result = await Process.run(
       "go",

lib/controllers/author_controller.dart

@@ -2,6 +2,7 @@ import "dart:async";
 import "package:collection/collection.dart";
 import "package:fast_immutable_collections/fast_immutable_collections.dart";
 import "package:flutter_riverpod/flutter_riverpod.dart";
+import "package:nexus/controllers/client_state_controller.dart";
 import "package:nexus/controllers/members_controller.dart";
 import "package:nexus/models/configs/author_config.dart";
 import "package:nexus/models/membership.dart";
@@ -12,7 +13,7 @@ class AuthorController extends AsyncNotifier<Membership> {
 
   @override
   Future<Membership> build() async {
-    var member = await ref.watch(
+    final member = await ref.watch(
       MembersController.provider(config.room).selectAsync(
         (value) => value.firstWhereOrNull(
           (membership) => membership.userId == config.message.authorId,
@@ -25,6 +26,12 @@ class AuthorController extends AsyncNotifier<Membership> {
         : Membership.fromContent(
             IMap(config.message.metadata?["pmp"]),
             config.message.authorId,
+            ref.watch(
+                  ClientStateController.provider.select(
+                    (value) => value?.homeserverUrl,
+                  ),
+                ) ??
+                "",
           );
 
     return Membership(

lib/controllers/members_controller.dart

@@ -1,6 +1,7 @@
 import "package:fast_immutable_collections/fast_immutable_collections.dart";
 import "package:flutter_riverpod/flutter_riverpod.dart";
 import "package:nexus/controllers/client_controller.dart";
+import "package:nexus/controllers/client_state_controller.dart";
 import "package:nexus/models/membership.dart";
 import "package:nexus/models/requests/get_room_state_request.dart";
 import "package:nexus/models/room.dart";
@@ -26,8 +27,16 @@ class MembersController extends AsyncNotifier<IList<Membership>> {
     return state.nonNulls
         .where((member) => member.content["membership"] == "join")
         .map(
-          (membership) =>
-              Membership.fromContent(membership.content, membership.stateKey!),
+          (membership) => Membership.fromContent(
+            membership.content,
+            membership.stateKey!,
+            ref.watch(
+                  ClientStateController.provider.select(
+                    (value) => value?.homeserverUrl,
+                  ),
+                ) ??
+                "",
+          ),
         )
         .toIList();
   }

lib/controllers/rooms_controller.dart

@@ -1,7 +1,9 @@
 import "package:collection/collection.dart";
 import "package:fast_immutable_collections/fast_immutable_collections.dart";
 import "package:flutter_riverpod/flutter_riverpod.dart";
+import "package:nexus/controllers/client_state_controller.dart";
 import "package:nexus/controllers/new_events_controller.dart";
+import "package:nexus/helpers/extensions/mxc_to_https.dart";
 import "package:nexus/models/read_receipt.dart";
 import "package:nexus/models/room.dart";
 
@@ -10,6 +12,13 @@ class RoomsController extends Notifier<IMap<String, Room>> {
   IMap<String, Room> build() => const IMap.empty();
 
   void update(IMap<String, Room> rooms, ISet<String> leftRooms) {
+    final homeserver =
+        ref.watch(
+          ClientStateController.provider.select(
+            (value) => value?.homeserverUrl,
+          ),
+        ) ??
+        "";
     final merged = rooms.entries.fold(state, (acc, entry) {
       final roomId = entry.key;
       final incoming = entry.value;
@@ -37,7 +46,13 @@ class RoomsController extends Notifier<IMap<String, Room>> {
         roomId,
         existing?.copyWith(
               hasMore: incoming.hasMore,
-              metadata: incoming.metadata ?? existing.metadata,
+              metadata:
+                  incoming.metadata?.copyWith(
+                    avatar:
+                        incoming.metadata?.avatar?.mxcToHttps(homeserver) ??
+                        existing.metadata?.avatar,
+                  ) ??
+                  existing.metadata,
               events: events!,
               state: incoming.state.entries.fold(
                 existing.state,
@@ -67,7 +82,11 @@ class RoomsController extends Notifier<IMap<String, Room>> {
                 ),
               ),
             ) ??
-            incoming,
+            incoming.copyWith(
+              metadata: incoming.metadata?.copyWith(
+                avatar: incoming.metadata?.avatar?.mxcToHttps(homeserver),
+              ),
+            ),
       );
     });
 

lib/controllers/spaces_controller.dart

@@ -1,3 +1,4 @@
+import "package:collection/collection.dart";
 import "package:fast_immutable_collections/fast_immutable_collections.dart";
 import "package:flutter/material.dart";
 import "package:flutter_riverpod/flutter_riverpod.dart";
@@ -99,15 +100,37 @@ class SpacesController extends Notifier<IList<Space>> {
         .toIList();
 
     return <Space>[
-      Space(id: "home", title: "Home", icon: Icons.home, children: homeRooms),
-      Space(
-        id: "dms",
-        title: "Direct Messages",
-        icon: Icons.people,
-        children: dmRooms,
-      ),
-      ...topLevelSpacesList,
-    ].toIList();
+          Space(
+            id: "home",
+            title: "Home",
+            icon: Icons.home,
+            children: homeRooms,
+          ),
+          Space(
+            id: "dms",
+            title: "Direct Messages",
+            icon: Icons.people,
+            children: dmRooms,
+          ),
+          ...topLevelSpacesList,
+        ]
+        .map(
+          (space) => space.copyWith(
+            children: space.children
+                .sortedBy(
+                  (element) =>
+                      element
+                          .metadata
+                          ?.sortingTimestamp
+                          .millisecondsSinceEpoch ??
+                      0,
+                )
+                .sortedBy((room) => room.metadata?.unreadMessages ?? 0)
+                .reversed
+                .toIList(),
+          ),
+        )
+        .toIList();
   }
 
   static final provider = NotifierProvider<SpacesController, IList<Space>>(

lib/helpers/extensions/mxc_to_https.dart

@@ -1,4 +1,5 @@
 extension MxcToHttps on Uri {
-  Uri mxcToHttps(String homeserver) =>
-      Uri.parse("${homeserver}_matrix/client/v1/media/download/$host$path");
+  Uri mxcToHttps(String homeserver) => Uri.parse(
+    homeserver,
+  ).resolve("_matrix/client/v1/media/download/$host$path");
 }

lib/models/membership.dart

@@ -1,5 +1,6 @@
 import "package:fast_immutable_collections/fast_immutable_collections.dart";
 import "package:freezed_annotation/freezed_annotation.dart";
+import "package:nexus/helpers/extensions/mxc_to_https.dart";
 part "membership.freezed.dart";
 
 @freezed
@@ -14,8 +15,11 @@ abstract class Membership with _$Membership {
   factory Membership.fromContent(
     IMap<String, dynamic> content,
     String userId,
+    String homeserver,
   ) => Membership(
-    avatarUrl: Uri.tryParse(content["avatar_url"] ?? ""),
+    avatarUrl: Uri.tryParse(
+      content["avatar_url"] ?? "",
+    )?.mxcToHttps(homeserver),
     userId: userId,
     displayName: content["displayname"] ?? userId.substring(1).split(":").first,
   );

lib/widgets/avatar_or_hash.dart

@@ -2,10 +2,8 @@ import "package:color_hash/color_hash.dart";
 import "package:cross_cache/cross_cache.dart";
 import "package:flutter/material.dart";
 import "package:flutter_riverpod/flutter_riverpod.dart";
-import "package:nexus/controllers/client_state_controller.dart";
 import "package:nexus/controllers/cross_cache_controller.dart";
 import "package:nexus/helpers/extensions/get_headers.dart";
-import "package:nexus/helpers/extensions/mxc_to_https.dart";
 
 class AvatarOrHash extends ConsumerWidget {
   final Uri? avatar;
@@ -48,16 +46,7 @@ class AvatarOrHash extends ConsumerWidget {
                   ? fallback ?? box
                   : Image(
                       image: CachedNetworkImage(
-                        avatar!
-                            .mxcToHttps(
-                              ref.watch(
-                                    ClientStateController.provider.select(
-                                      (value) => value?.homeserverUrl,
-                                    ),
-                                  ) ??
-                                  "",
-                            )
-                            .toString(),
+                        avatar.toString(),
                         ref.watch(CrossCacheController.provider),
                         headers: ref.headers,
                       ),

lib/widgets/chat_page/expandable_image.dart

@@ -0,0 +1,48 @@
+import "dart:math";
+import "package:cross_cache/cross_cache.dart";
+import "package:flutter/material.dart";
+import "package:hooks_riverpod/hooks_riverpod.dart";
+import "package:nexus/controllers/cross_cache_controller.dart";
+import "package:nexus/helpers/extensions/get_headers.dart";
+import "package:nexus/widgets/error_dialog.dart";
+
+class ExpandableImage extends ConsumerWidget {
+  final Widget child;
+  final String? source;
+  const ExpandableImage(this.source, {required this.child, super.key});
+
+  @override
+  Widget build(BuildContext context, WidgetRef ref) => InkWell(
+    onTap: source == null
+        ? null
+        : () => showDialog(
+            context: context,
+            builder: (_) => LayoutBuilder(
+              builder: (context, constraints) => Dialog(
+                backgroundColor: Colors.transparent,
+                insetPadding: EdgeInsets.all(constraints.maxWidth / 100),
+                child: ConstrainedBox(
+                  constraints: BoxConstraints(
+                    minWidth: min(constraints.maxWidth, 1000),
+                  ),
+                  child: InteractiveViewer(
+                    child: Image(
+                      fit: BoxFit.contain,
+                      errorBuilder: (_, error, stackTrace) => ErrorDialog(
+                        "Loading failed for $source\nError: $error",
+                        stackTrace,
+                      ),
+                      image: CachedNetworkImage(
+                        source!,
+                        ref.watch(CrossCacheController.provider),
+                        headers: ref.headers,
+                      ),
+                    ),
+                  ),
+                ),
+              ),
+            ),
+          ),
+    child: child,
+  );
+}

lib/widgets/chat_page/expandable_image_message.dart

@@ -1,4 +1,3 @@
-import "dart:math";
 import "package:cross_cache/cross_cache.dart";
 import "package:flutter/material.dart";
 import "package:flutter_chat_core/flutter_chat_core.dart";
@@ -6,6 +5,7 @@ import "package:flutter_riverpod/flutter_riverpod.dart";
 import "package:flyer_chat_image_message/flyer_chat_image_message.dart";
 import "package:nexus/controllers/cross_cache_controller.dart";
 import "package:nexus/helpers/extensions/get_headers.dart";
+import "package:nexus/widgets/chat_page/expandable_image.dart";
 
 class ExpandableImageMessage extends ConsumerWidget {
   final ImageMessage message;
@@ -14,31 +14,8 @@ class ExpandableImageMessage extends ConsumerWidget {
   const ExpandableImageMessage(this.message, {required this.index, super.key});
 
   @override
-  Widget build(BuildContext context, WidgetRef ref) => InkWell(
-    onTap: () => showDialog(
-      context: context,
-      builder: (_) => LayoutBuilder(
-        builder: (context, constraints) => Dialog(
-          backgroundColor: Colors.transparent,
-          insetPadding: EdgeInsets.all(constraints.maxWidth / 100),
-          child: ConstrainedBox(
-            constraints: BoxConstraints(
-              minWidth: min(constraints.maxWidth, 1000),
-            ),
-            child: InteractiveViewer(
-              child: Image(
-                fit: BoxFit.contain,
-                image: CachedNetworkImage(
-                  message.source,
-                  ref.watch(CrossCacheController.provider),
-                  headers: ref.headers,
-                ),
-              ),
-            ),
-          ),
-        ),
-      ),
-    ),
+  Widget build(BuildContext context, WidgetRef ref) => ExpandableImage(
+    message.source,
     child: FlyerChatImageMessage(
       customImageProvider: CachedNetworkImage(
         message.source,

lib/widgets/chat_page/room_appbar.dart

@@ -3,6 +3,7 @@ import "package:flutter/material.dart";
 import "package:nexus/models/room.dart";
 import "package:nexus/widgets/appbar.dart";
 import "package:nexus/widgets/avatar_or_hash.dart";
+import "package:nexus/widgets/chat_page/expandable_image.dart";
 import "package:nexus/widgets/chat_page/room_menu.dart";
 
 class RoomAppbar extends StatelessWidget implements PreferredSizeWidget {
@@ -24,11 +25,14 @@ class RoomAppbar extends StatelessWidget implements PreferredSizeWidget {
   @override
   Widget build(BuildContext context) => Appbar(
     leading: isDesktop
-        ? AvatarOrHash(
-            room.metadata?.avatar,
-            room.metadata?.name ?? "Unnamed Rooms",
-            height: 24,
-            fallback: Icon(Icons.numbers),
+        ? ExpandableImage(
+            room.metadata?.avatar?.toString(),
+            child: AvatarOrHash(
+              room.metadata?.avatar,
+              room.metadata?.name ?? "Unnamed Rooms",
+              height: 24,
+              fallback: Icon(Icons.numbers),
+            ),
           )
         : DrawerButton(onPressed: () => onOpenDrawer(context)),
     scrolledUnderElevation: 0,

lib/widgets/chat_page/room_chat.dart

@@ -14,7 +14,7 @@ import "package:nexus/helpers/extensions/show_context_menu.dart";
 import "package:nexus/models/relation_type.dart";
 import "package:nexus/models/requests/report_request.dart";
 import "package:nexus/widgets/chat_page/composer/chat_box.dart";
-import "package:nexus/widgets/chat_page/image_message.dart";
+import "package:nexus/widgets/chat_page/expandable_image_message.dart";
 import "package:nexus/widgets/chat_page/member_list.dart";
 import "package:nexus/widgets/chat_page/wrappers/message_wrapper.dart";
 import "package:nexus/widgets/chat_page/room_appbar.dart";

linux/runner/my_application.cc

@@ -43,7 +43,7 @@ static void my_application_activate(GApplication* application) {
     }
   }
 #endif
-  gtk_widget_set_size_request(GTK_WIDGET(window), 500, 500);
+  gtk_widget_set_size_request(GTK_WIDGET(window), 250, -1);
   if (use_header_bar) {
     GtkHeaderBar* header_bar = GTK_HEADER_BAR(gtk_header_bar_new());
     gtk_widget_show(GTK_WIDGET(header_bar));