fix: 2 bugfixes (brand not removable + minor issue) (#6199)

* Always unfocus before showing the "May exit…" sheet

* Fix: unable to remove reordered brands

Author: g123k

packages/smooth_app/lib/pages/product/add_basic_details_page.dart

@@ -61,7 +61,6 @@ class _AddBasicDetailsPageState extends State<AddBasicDetailsPage> {
   final SimpleInputPageBrandsHelper _brandsHelper =
       SimpleInputPageBrandsHelper();
   late final MultilingualHelper _multilingualHelper;
-  late final FocusNode _focusNode;
 
   /// Only used when there's not enough place
   bool _showPhotosBanner = false;
@@ -81,7 +80,6 @@ class _AddBasicDetailsPageState extends State<AddBasicDetailsPage> {
       monolingualText: _product.productName,
       productLanguage: _product.lang,
     );
-    _focusNode = FocusNode();
   }
 
   @override
@@ -137,7 +135,6 @@ class _AddBasicDetailsPageState extends State<AddBasicDetailsPage> {
     _productNameController.dispose();
     _brandsController.dispose();
     _weightController.dispose();
-    _focusNode.dispose();
     super.dispose();
   }
 

packages/smooth_app/lib/pages/product/may_exit_page_helper.dart

@@ -18,7 +18,12 @@ class MayExitPageHelper {
   Future<bool?> openSaveBeforeLeavingDialog(
     final BuildContext context, {
     final String? title,
+    final bool unfocus = true,
   }) async {
+    if (unfocus) {
+      FocusManager.instance.primaryFocus?.unfocus();
+    }
+
     final AppLocalizations appLocalizations = AppLocalizations.of(context);
     final SmoothColorsThemeExtension extension =
         context.extension<SmoothColorsThemeExtension>();

packages/smooth_app/lib/pages/product/simple_input_page_helpers.dart

@@ -207,7 +207,7 @@ abstract class AbstractSimpleInputPageHelper extends ChangeNotifier {
 
   /// Mainly used when reordering the list.
   void replaceItems(final List<String> items) {
-    _terms = items;
+    _terms = List<String>.of(items);
     _changed = true;
     notifyListeners();
   }