Landing page

Author: aothms

frontend/src/App.js

@@ -125,15 +125,18 @@ function App() {
       .then(response => response.json())
       .then((data) => {
         if (data["redirect"] !== undefined && data["redirect"] !== null) {
+          /*
+          // In Dashboard.js we do redirect, but not here
+          // @todo clean-up some duplication in the react code
           if (!window.location.href.endsWith(data.redirect)) {
             window.location.href = data.redirect;
           }
+          */
         }
         else {
           setLogin(true);
           setUser(data["user_data"]);
           data["sandbox_info"]["pr_title"] && setPrTitle(data["sandbox_info"]["pr_title"]);
-
         }
       })
   }, []);
@@ -238,7 +241,7 @@ function App() {
                     <div style={{ background: '#D9D9D9', padding: '5em', marginTop: '5em' }}>
                       <Typography variant='h5' style={{ fontWeight: 'bold' }}>Validate your IFC files against the standard</Typography>
                       <Typography style={{ margin: '1em 0 2em 0' }}>A free, online platform by buildingSMART for checking IFC file conformity against STEP syntax, IFC schema, and normative specification rules.</Typography>
-                      <Link sx={{fontWeight: 700, textDecoration: 'none', color: '#000', '&:hover': {borderBottom: 'dotted 1px black', color: '#444'}}} href='/dashboard'>Start validating →</Link>
+                      <Link sx={{fontWeight: 700, textDecoration: 'none', color: '#000', '&:hover': {borderBottom: 'dotted 1px black', color: '#333'}}} href='/dashboard'>Start validating →</Link>
                     </div>
 
                     <Typography variant='h6'>How it checks</Typography>

frontend/src/ResponsiveAppBar.js

@@ -75,53 +75,60 @@ function ResponsiveAppBar({ user }) {
           <Box sx={{ flexGrow: 1 }} />
 
           {user &&
-            <>
-              <Box sx={{ flexGrow: 0 }}>
-                <Tooltip title="Navigation">
-                  <IconButton onClick={handleOpenUserMenu} sx={{ p: 0 }}>
-                    <Avatar
-                      sx={{
-                        bgcolor: "grey",
-                        "&:hover": {
-                          border: "2px solid darkgrey",
-                        },
-                      }}
-                    >
-                      {user ? (Array.from(user["given_name"])[0] + Array.from(user["family_name"])[0]) : ""}
-                    </Avatar>
-                  </IconButton>
-                </Tooltip>
-                <Menu
-                  sx={{ mt: '45px' }}
-                  id="menu-appbar"
-                  anchorEl={anchorElUser}
-                  anchorOrigin={{
-                    vertical: 'top',
-                    horizontal: 'right',
-                  }}
-                  keepMounted
-                  transformOrigin={{
-                    vertical: 'top',
-                    horizontal: 'right',
-                  }}
-                  open={Boolean(anchorElUser)}
-                  onClose={handleCloseUserMenu}
-                >
-                  {pages.map((p) => (
-                    <MenuItem key={p.label} onClick={handleCloseUserMenu}>
-                      <Link color="grey" href={p.href} underline="none">
-                        <Typography textAlign="center">{p.label}</Typography>
-                      </Link>
-                    </MenuItem>
-                  ))}
-                </Menu>
-              </Box>
-            </>
+            <Box sx={{ flexGrow: 0 }}>
+              <Tooltip title="Navigation">
+                <IconButton onClick={handleOpenUserMenu} sx={{ p: 0 }}>
+                  <Avatar
+                    sx={{
+                      bgcolor: "grey",
+                      "&:hover": {
+                        border: "2px solid darkgrey",
+                      },
+                    }}
+                  >
+                    {user ? (Array.from(user["given_name"])[0] + Array.from(user["family_name"])[0]) : ""}
+                  </Avatar>
+                </IconButton>
+              </Tooltip>
+              <Menu
+                sx={{ mt: '45px' }}
+                id="menu-appbar"
+                anchorEl={anchorElUser}
+                anchorOrigin={{
+                  vertical: 'top',
+                  horizontal: 'right',
+                }}
+                keepMounted
+                transformOrigin={{
+                  vertical: 'top',
+                  horizontal: 'right',
+                }}
+                open={Boolean(anchorElUser)}
+                onClose={handleCloseUserMenu}
+              >
+                {pages.map((p) => (
+                  <MenuItem key={p.label} onClick={handleCloseUserMenu}>
+                    <Link color="grey" href={p.href} underline="none">
+                      <Typography textAlign="center">{p.label}</Typography>
+                    </Link>
+                  </MenuItem>
+                ))}
+              </Menu>
+            </Box>
           }
 
           {!user &&
-            <Box sx={{ flexGrow: 1 }}>
-              <Link href="/login">Sign In</Link>
+            <Box sx={{ flexGrow: 0 }}>
+              <Link sx={{
+                textDecoration: 'none',
+                color: 'black',
+                display: 'inline-block',
+                border: 'solid 1px black',
+                padding: '0.5em 1em',
+                borderRadius: '0.5em',
+                fontWeight: 'bold',
+                '&:hover': {textDecoration: 'underline', color: '#333'}
+              }} href="/login">Sign In</Link>
             </Box>
           }
         </Toolbar>