fixed #167

Author: Longwater1234

TODO.md

@@ -2,7 +2,7 @@
 
 - [ ] (Online mode) Display countdown timer (40 s), waiting for player's move. Reset it when they make a move.
 - [ ] Record and list all previous moves and captures (for current match), in a scroll panel.
-- [ ] highlight cell color blue after clicking a piece, reset when click srcCell = null
+- [x] highlight cell color blue after clicking a piece, reset when click srcCell = null
 - [ ] write unit tests using googletests
 
 ### (Extras) Standard CJK font paths

src/Cell.hpp

@@ -125,13 +125,15 @@ const inline sf::Vector2f &Cell::getPos() const
  */
 inline void Cell::highlightActive()
 {
+    this->rec.setFillColor(BABY_BLUE);
 }
 
 /**
  * Restore the original color
  */
 inline void Cell::resetColor()
 {
+    this->rec.setFillColor(DARK_BROWN);
 }
 
 } // namespace chk

src/managers/GameManager.cpp

@@ -296,6 +296,13 @@ void chk::GameManager::handleCellTap(const chk::PlayerPtr &hunter, const chk::Pl
     {
         return;
     }
+    // reset color of all previous active cells
+    std::for_each(this->blockList.begin(), this->blockList.end(), [](const chk::Block &cell) {
+        if (cell->getIndex() != -1)
+        {
+            cell->resetColor();
+        }
+    });
     // CHECK IF this cell has a Piece
     const short pieceId = this->getPieceFromCell(cell->getIndex());
     if (pieceId != -1)
@@ -309,15 +316,6 @@ void chk::GameManager::handleCellTap(const chk::PlayerPtr &hunter, const chk::Pl
         }
         // OTHERWISE, store it in buffer (for a SIMPLE/CAPTURE move next)!
         buffer.addItem(pieceId);
-        // copy original src cell
-        int copySrcCell = this->sourceCell.value();
-        // reset old active cell color
-        const auto it = std::find_if(this->blockList.begin(), this->blockList.end(),
-                               [&](const chk::Block &cell) { cell->getIndex() == copySrcCell; });
-        if (it != this->blockList.end())
-        {
-         
-        }
         this->setSourceCell(cell->getIndex());
         cell->highlightActive();
     }

src/managers/OnlineGameManager.hpp

@@ -398,7 +398,13 @@ inline void OnlineGameManager::handleCellTap(const chk::PlayerPtr &hunter, const
     {
         return;
     }
-
+    // reset color of all previous active cells
+    std::for_each(this->blockList.begin(), this->blockList.end(), [](const chk::Block &cell) {
+        if (cell->getIndex() != -1)
+        {
+            cell->resetColor();
+        }
+    });
     // CHECK IF this cell has a Piece
     const short pieceId = this->getPieceFromCell(cell->getIndex());
     if (pieceId != -1)
@@ -413,6 +419,7 @@ inline void OnlineGameManager::handleCellTap(const chk::PlayerPtr &hunter, const
         // OTHERWISE, store it in buffer (for a simple move, on next turn)
         buffer.addItem(pieceId);
         this->setSourceCell(cell->getIndex());
+        cell->highlightActive();
     }
     else
     {