tasx
February 27, 2025, 9:43pm
1
I don’t know how the grid appears for everyone else, but this is what i see:
The text inside the grid is too long and wraps around. It should be safe to remove it altogether without any negative impact. Another approach would be to enclose the potential tree in brackets, like this: [5]
tasx
February 27, 2025, 9:57pm
2
1 2 3 4
|-----------
1 | 9 8 7 8
2 |[5] 3 2 4
3 | 6 6 7 1
or
↓
1 2 3 4
|-----------
1 | 9 8 7 8
→ 2 |[5] 3 2 4
3 | 6 6 7 1
IsaacG
February 27, 2025, 11:00pm
3
Option two looks great to me!
tasx
March 7, 2025, 5:09pm
4
Let’s go with option 2 then
exercism:main
← tasxatzial:saddle-points-grid-fix
opened 05:09PM - 07 Mar 25 UTC
Forum
https://forum.exercism.org/t/saddle-points-issue-with-grid-display/1604… 4
Summary: Removed the text inside the code block to prevent issues with text wrapping and marked the saddle point differently. The explanation below the grid is already sufficient to clarify what is happening.
2 Likes