Working fairly well

Signed-off-by: James Ketrenos <james_git@ketrenos.com>

client/src/App.css

@@ -19,12 +19,19 @@ div {
   height: 100%;
 }
 
-.Resizer {
+.Explorer .Resizer {
   width: 0.5rem;
   background-color: #ccc;
   border: 1px solid black;
 }
 
+.ExplorerVertical .Resizer {
+  height: 0.5rem;
+  width: 100%;
+  background-color: #ccc;
+  border: 1px solid black;
+}
+
 .Explorer {
   display: flex;
   flex-direction: row;
@@ -35,6 +42,16 @@ div {
   flex-grow: 1;
 }
 
+.ExplorerVertical {
+  display: flex;
+  flex-direction: column;
+  justify-self: stretch;
+  align-self: stretch;
+  width: 100%;
+  height: auto !important;
+  flex-grow: 1;
+}
+
 .Actions {
   display: flex;
   flex-direction: row;
@@ -48,7 +65,6 @@ div {
 .Identities {
   display: grid;
   user-select: none;
-  overflow-y: scroll;
   overflow-x: clip;
   width: 100%;
   max-height: 100%; /* scroll if too large */
@@ -61,6 +77,7 @@ div {
 
 .Identities {
   grid-template-columns: repeat(auto-fill, minmax(4.25rem, auto));
+  overflow-y: scroll;
 }
 
 .Face {
@@ -81,18 +98,26 @@ div {
   flex-direction: column;
 }
 
-.Viewer {
-  display: flex;
-  flex-direction: column;
-  position: relative;
-  height: 100%;
-}
-
 .PhotoPanel {
   display: flex;
   flex-direction: column;
   justify-content: center;
   max-height: 100%;
+  overflow-y: auto;
+}
+
+.PhotoPanel .FaceInfo {
+  padding: 0.25rem;
+  background-color: #444;
+  color: white;
+  margin-top: 0.25rem;
+}
+
+.PhotoPanel .ImageInfo {
+  padding: 0.25rem;
+  background-color: #222;
+  color: white;
+  margin-top: 0.25rem;
 }
 
 .Guess {
@@ -130,19 +155,7 @@ button {
 .Photo {
   display: flex;
   position: relative;
-}
+  flex-direction: column;
-
-.PhotoPanel .FaceInfo {
-  padding: 0.25rem;
-  background-color: #444;
-  color: white;
-  margin-top: 0.25rem;}
-
-.PhotoPanel .ImageInfo {
-  padding: 0.25rem;
-  background-color: #222;
-  color: white;
-  margin-top: 0.25rem;
 }
 
 .PhotoFaces .UnknownFace,

client/src/App.tsx

@@ -118,7 +118,6 @@ const Photo = ({ photoId, onFaceClick }: any) => {
     }
   }, [setDimensions, dimensions]);
 
- 
   useEffect(() => {
     let timer = setInterval(() => checkResize(), 250);
     return () => { clearInterval(timer); }
@@ -152,6 +151,9 @@ const Photo = ({ photoId, onFaceClick }: any) => {
       });
       await res.json();
       setPhotoSelected([]);
+      photo.faces = photo.faces.filter(
+        face => photoSelected.findIndex(x => x.faceId === face.faceId) === -1
+      );
       setPhoto({...photo});
     } catch (error) {
       console.error(error);
@@ -196,7 +198,8 @@ const Photo = ({ photoId, onFaceClick }: any) => {
     return <></>
   }
 
-  return (<div className="PhotoPanel">
+  return (
+  <div className="PhotoPanel">
     <div className="Photo" ref={ref}>
       <img 
         alt={photo.filename}
@@ -217,7 +220,9 @@ const Photo = ({ photoId, onFaceClick }: any) => {
         <Button onClick={selectAll}>Select All</Button>
         <Button onClick={clearSelection}>Select None</Button>
       </div>
-      { photoSelected.length !== 0 && <div>Selected faces (CTRL-CLICK to remove):</div> }
+      { photoSelected.length !== 0 && <div style={{ display: 'flex' }}>
+        Selected faces (CTRL-CLICK to remove):
+      </div> }
       <div className="PhotoFaces">
       { photoSelected.map(face =>
         <Face
@@ -1046,10 +1051,13 @@ const App = () => {
             </div>
           </Panel>
           <PanelResizeHandle className="Resizer"/>
-          <Panel>
+          <Panel><>
-            <div className="Viewer">
+            { image === 0 && 
-              {image === 0 && <div style={{ margin: '1rem' }}>Select image to view</div>}
+              <div style={{ margin: '1rem' }}>Select image to view</div>
-              {image !== 0 && <Photo onFaceClick={onFaceClick} photoId={image}/> }
+            }
+            { image !== 0 && 
+              <Photo onFaceClick={onFaceClick} photoId={image}/> 
+            }
             { guess !== undefined && guess.identity && <div 
               className="Guess">
               <Face
@@ -1057,8 +1065,7 @@ const App = () => {
                 onFaceClick={guessOnFaceClick}
                 title={`${guess.identity.displayName} (${guess.distance})`}/>
             </div> }
-            </div>
+          </></Panel>
-          </Panel>
           <PanelResizeHandle className="Resizer" />
           <Panel defaultSize={8.5} minSize={8.5} className="IdentitiesList">
             { !loaded && <div style={{ margin: '1rem' }}>

server/routes/identities.js

@@ -448,6 +448,10 @@ router.put("/faces/add/:id", async (req, res) => {
     /* Do not block on this call finishing -- update can occur
      * in the background */
     Promise.map([identity, ...tuples], (x, i) => { 
+      if (x.identityId === null) {
+        /* Moving from the Unknown group */
+        return;
+      }
       try {
         updateIdentityFaces(x);
       } catch (error) {

server/routes/photos.js

@@ -1109,6 +1109,7 @@ router.get("/:id", async (req, res) => {
       `
       SELECT faces.* FROM faces
       WHERE faces.photoId=:id
+      AND faces.classifiedBy NOT IN ('not-a-face', 'forget' )
       `, {
         replacements: { id },
         type: photoDB.Sequelize.QueryTypes.SELECT