Merge pull request #162 from jpd236/shift-arrow-regression

Fix regression in shift-arrow behavior on filled words.
mrichards42 · Aug 18, 2021 · e5609e1 · e5609e1
2 parents d76109b + 5093ec1
commit e5609e1
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/XGridCtrl.cpp b/src/XGridCtrl.cpp
@@ -1656,7 +1656,8 @@ XGridCtrl::OnArrow(puz::GridDirection arrowDirection, int mod)
     else // Shift
     {
         // If the arrow is in the focused direction, wrap around the grid.
-        unsigned int options = AreInLine(m_focusedDirection, arrowDirection) ? 0 : puz::NO_WRAP;
+        bool sameDirection = AreInLine(m_focusedDirection, arrowDirection);
+        unsigned int options = sameDirection ? 0 : puz::NO_WRAP;
         puz::GridDirection focusedDirection = 
             static_cast<puz::GridDirection>(m_focusedDirection);
         // Move to the next white square in the arrow direction that
@@ -1718,8 +1719,8 @@ XGridCtrl::OnArrow(puz::GridDirection arrowDirection, int mod)
                 newWord = closestSquareWord;
             }
         }
-        // Find the first square in the word
-        if (newSquare) {
+        // If the arrow is in the focused direction, find the first square in the word.
+        if (newSquare && sameDirection) {
             if (newWord)
                 newSquare = newWord->front();
             else