Changes in components/BuildDetailsDialog.tsx [f727252:41a2f81]

File:

• components/BuildDetailsDialog.tsx (modified) (9 diffs)

components/BuildDetailsDialog.tsx

@@ -1 +1 @@
 import React, {useEffect, useState} from 'react';
 import {
-    Dialog, DialogTitle, DialogContent, DialogActions, Button, Box, Typography,
+    Dialog, DialogTitle, DialogContent, DialogActions, Button, Grid, Box, Typography,
     IconButton, Tab, Tabs, Table, TableBody, TableCell, TableRow, Rating, TextField, Avatar, Chip, Alert
 } from '@mui/material';
@@ -34 +34 @@
     const [ratingVal, setRatingVal] = useState(5);
 
+    // Main details fetch
     useEffect(() => {
-        if (open && buildId !== null) {
+        if (open && buildId !== null && typeof buildId === 'number') {
             setLoading(true);
             setReviewText("");
@@ -51 +52 @@
 
     useEffect(() => {
-        if (open && buildId !== null) {
+        if (open && buildId !== null && typeof buildId === 'number') {
             onGetBuildState({buildId})
                 .then(state => {
@@ -62 +63 @@
     }, [open, buildId]);
 
-    useEffect(() => {
-        if (open) {
-            setTabIndex(0);
-        }
-    }, [open, buildId]);
-
-
     const handleFavorite = async () => {
         if (!currentUser || buildId === null) return alert("Please login to favorite builds.");
@@ -109 +103 @@
     return (
         <>
-            <Dialog open={open} onClose={onClose} maxWidth="md" fullWidth scroll="body">
+            <Dialog open={open} onClose={onClose} maxWidth="md" fullWidth scroll="paper">
                 {loading || !details ? (
                     <Box sx={{p: 5, textAlign: 'center'}}>Loading Forge Schematics...</Box>
@@ -152 +146 @@
                             <Box sx={{p: 3}}>
                                 {tabIndex === 0 && (
-                                    <Box
-                                        sx={{
-                                            display: 'grid',
-                                            gridTemplateColumns: {
-                                                xs: '1fr',
-                                                md: '2fr 1fr'
-                                            },
-                                            gap: 2,
-                                            width: '100%'
-                                        }}
-                                    >
-                                        <Box>
+                                    <Grid container spacing={2}>
+                                        <Grid item xs={12} md={8}>
                                             <Table size="small">
                                                 <TableBody>
@@ -209 +193 @@
                                                 </TableBody>
                                             </Table>
-                                        </Box>
-
-                                        <Box>
+                                        </Grid>
+
+                                        <Grid item xs={12} md={4}>
                                             <Box sx={{bgcolor: '#424343', p: 2, borderRadius: 2, mb: 2}}>
-                                                <Typography color="primary.main" gutterBottom fontWeight="bold">
-                                                    Builder's Notes
-                                                </Typography>
+                                                <Typography color="primary.main" gutterBottom fontWeight="bold">Builder's
+                                                    Notes</Typography>
                                                 <Typography color="primary.main" variant="body2"
                                                             sx={{fontStyle: 'italic'}}>
@@ -260 +243 @@
                                                 </Button>
                                             </Box>
-                                        </Box>
-                                    </Box>
+                                        </Grid>
+                                    </Grid>
                                 )}
 
@@ -330 +313 @@
                                             ))}
                                             {details.reviews.length === 0 && (
-                                                <Typography color="text.secondary" align="center">
-                                                    No reviews yet. Be the first!
-                                                </Typography>
+                                                <Typography color="text.secondary" align="center">No reviews yet. Be the
+                                                    first!</Typography>
                                             )}
                                         </Box>