Merge pull request #79 from owlistic-notes/fix-create-button

Fix create button

Author: DavideRutigliano

src/frontend/lib/screens/home_screen.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:go_router/go_router.dart';
+import 'package:owlistic/core/theme.dart';
 import 'package:owlistic/utils/data_converter.dart';
 import 'package:provider/provider.dart';
 import 'package:owlistic/models/user.dart';
@@ -604,7 +605,7 @@ class _HomeScreenState extends State<HomeScreen> {
                     selectedNotebookId!,
                   );
 
-                  Navigator.pop(context);
+                  // Navigator.pop(context);
 
                   if (note != null) {
                     // Navigate to the new note
@@ -624,6 +625,7 @@ class _HomeScreenState extends State<HomeScreen> {
                 }
               }
             },
+            style: AppTheme.getSuccessButtonStyle(),
             child: const Text('Create'),
           ),
         ],
@@ -634,6 +636,12 @@ class _HomeScreenState extends State<HomeScreen> {
   void _showAddTaskDialog(BuildContext context) {
     final titleController = TextEditingController();
 
+    // Use HomeViewModel for notebooks
+    final homeViewModel = context.read<HomeViewModel>();
+    final notes = homeViewModel.recentNotes;
+
+    String? selectedNoteId;
+
     showDialog(
       context: context,
       builder: (ctx) => AlertDialog(
@@ -647,13 +655,44 @@ class _HomeScreenState extends State<HomeScreen> {
         shape: RoundedRectangleBorder(
           borderRadius: BorderRadius.circular(16),
         ),
-        content: TextField(
-          controller: titleController,
-          decoration: const InputDecoration(
-            labelText: 'Task Title',
-            prefixIcon: Icon(Icons.title),
-          ),
-          autofocus: true,
+        content: Column(
+          mainAxisSize: MainAxisSize.min,
+          children: [
+            TextField(
+              controller: titleController,
+              decoration: const InputDecoration(
+                labelText: 'Task Title',
+                prefixIcon: Icon(Icons.title),
+              ),
+              autofocus: true,
+            ),
+            const SizedBox(height: 16),
+            DropdownButtonFormField<String>(
+              decoration: const InputDecoration(
+                labelText: 'Select Note',
+                prefixIcon: Icon(Icons.book),
+              ),
+              value: selectedNoteId,
+              items: [
+                if (notes.isEmpty)
+                  const DropdownMenuItem(
+                    value: '',
+                    child: Text('No notes available'),
+                  ),
+                ...notes.map(
+                  (note) => DropdownMenuItem(
+                    value: note.id,
+                    child: Text(note.title),
+                  ),
+                ),
+              ],
+              onChanged: (value) {
+                setState(() {
+                  selectedNoteId = value;
+                });
+              },
+            ),
+          ],
         ),
         actions: [
           TextButton(
@@ -681,6 +720,7 @@ class _HomeScreenState extends State<HomeScreen> {
                 }
               }
             },
+            style: AppTheme.getSuccessButtonStyle(),
             child: const Text('Create'),
           ),
         ],
@@ -762,6 +802,7 @@ class _HomeScreenState extends State<HomeScreen> {
                 }
               }
             },
+            style: AppTheme.getSuccessButtonStyle(),
             child: const Text('Create'),
           ),
         ],

src/frontend/lib/screens/notebooks_screen.dart

@@ -1,4 +1,5 @@
 import 'package:flutter/material.dart';
+import 'package:owlistic/core/theme.dart';
 import 'package:provider/provider.dart';
 import 'package:owlistic/models/notebook.dart';
 import 'package:owlistic/models/note.dart';
@@ -105,7 +106,7 @@ class _NotebooksScreenState extends State<NotebooksScreen> {
         icon: Icons.book,
         title: 'No Notebooks',
         message: 'Create your first notebook to start organizing your notes.',
-        actionLabel: 'Create Notebook',
+        actionLabel: 'Add Notebook',
         onAction: () => _showAddNotebookDialog(context),
       );
     }
@@ -399,9 +400,7 @@ class _NotebooksScreenState extends State<NotebooksScreen> {
                 }
               }
             },
-            style: ElevatedButton.styleFrom(
-              backgroundColor: Theme.of(context).primaryColor,
-            ),
+            style: AppTheme.getSuccessButtonStyle(),
             child: const Text('Create'),
           ),
         ],
@@ -415,33 +414,44 @@ class _NotebooksScreenState extends State<NotebooksScreen> {
 
     showDialog(
       context: context,
-      builder: (context) => AlertDialog(
-        title: const Text('Create Notebook'),
+      builder: (ctx) => AlertDialog(
+        shape: RoundedRectangleBorder(
+          borderRadius: BorderRadius.circular(16),
+        ),
+        title: Row(
+          children: [
+            Icon(Icons.create_new_folder, color: Theme.of(context).primaryColor),
+            const SizedBox(width: 8),
+            const Text('Add Notebook'),
+          ],
+        ),
         content: Column(
           mainAxisSize: MainAxisSize.min,
           children: [
             TextField(
               controller: nameController,
               decoration: const InputDecoration(
-                labelText: 'Name',
-                hintText: 'Enter notebook name',
+                labelText: 'Notebook Name',
+                prefixIcon: Icon(Icons.book),
               ),
               autofocus: true,
             ),
             const SizedBox(height: 16),
             TextField(
               controller: descriptionController,
               decoration: const InputDecoration(
-                labelText: 'Description (optional)',
-                hintText: 'Enter notebook description',
+                labelText: 'Description (Optional)',
+                prefixIcon: Icon(Icons.description),
               ),
-              maxLines: 3,
+              maxLines: 2,
             ),
           ],
         ),
         actions: [
           TextButton(
-            onPressed: () => Navigator.pop(context),
+            onPressed: () {
+              Navigator.pop(context);
+            },
             child: const Text('Cancel'),
           ),
           ElevatedButton(
@@ -476,6 +486,7 @@ class _NotebooksScreenState extends State<NotebooksScreen> {
                 }
               }
             },
+            style: AppTheme.getSuccessButtonStyle(),
             child: const Text('Create'),
           ),
         ],