fix: use proper navigator when popping

Author: Quirille

packages/flutter_catalog/lib/src/views/catalog_modify_view.dart

@@ -110,11 +110,11 @@ class CatalogModifyView extends HookWidget {
           content: Text(localizations.itemCreatePageDeleteConfirmationMessage),
           actions: [
             TextButton(
-              onPressed: () => navigator.pop(false),
+              onPressed: () => Navigator.of(context).pop(false),
               child: Text(localizations.itemCreatePageDeleteConfirmationCancel),
             ),
             TextButton(
-              onPressed: () => navigator.pop(true),
+              onPressed: () => Navigator.of(context).pop(true),
               child: Text(
                 localizations.itemCreatePageDeleteConfirmationConfirm,
                 style: TextStyle(color: Theme.of(context).colorScheme.error),

packages/flutter_catalog/lib/src/widgets/catalog_grid_item.dart

@@ -33,31 +33,32 @@ class CatalogGridItem extends StatelessWidget {
       onTap: onTap,
       child: Card(
         clipBehavior: Clip.antiAlias,
-        shape: RoundedRectangleBorder(
-          borderRadius: BorderRadius.circular(16.0),
-        ),
-        elevation: 0,
-        color: colorScheme.surface,
         child: Column(
           crossAxisAlignment: CrossAxisAlignment.stretch,
           children: [
             if (item.imageUrls.isNotEmpty) ...[
               /// The item image, which expands to fill the available space.
               Expanded(
-                child: CachedNetworkImage(
-                  imageUrl: item.imageUrls.first,
-                  fit: BoxFit.cover,
-                  placeholder: (context, url) =>
-                      const Center(child: CircularProgressIndicator.adaptive()),
-                  errorWidget: (context, url, error) =>
-                      const Icon(Icons.broken_image),
+                child: Padding(
+                  padding: EdgeInsets.all(6.0),
+                  child: ClipRRect(
+                    borderRadius: BorderRadius.circular(12),
+                    child: CachedNetworkImage(
+                      imageUrl: item.imageUrls.first,
+                      fit: BoxFit.cover,
+                      placeholder: (context, url) => const Center(
+                          child: CircularProgressIndicator.adaptive()),
+                      errorWidget: (context, url, error) =>
+                          const Icon(Icons.broken_image),
+                    ),
+                  ),
                 ),
               ),
             ],
 
             /// The details section at the bottom of the card.
             Padding(
-              padding: const EdgeInsets.all(8.0),
+              padding: const EdgeInsets.all(8.0).copyWith(bottom: 16.0),
               child: Column(
                 crossAxisAlignment: CrossAxisAlignment.start,
                 children: [

packages/flutter_catalog_rest_api/lib/src/rest_catalog_repository.dart

@@ -167,6 +167,7 @@ class RestCatalogRepository<T extends CatalogItem>
   Future<void> deleteCatalogItem(String itemId) async {
     var deleteEndpoint = _baseEndpoint
         .child(deleteCatalogItemEndpoint)
+        .withConverter(NoOpConverter())
         .authenticate()
         .withVariables({"id": itemId});