Closing "Map Settings" dialog causes an exception. #1
Comments
cfcohen
added a commit
that referenced
this issue
Mar 27, 2021
This change should not be committed permanently, because it's actually much _worse_ on Linux than the code currently on master, but I'm curious to know if it works in Windows. It should look very much like the original behavior and should not require the Windows specific Wingdings font.
cfcohen
added a commit
that referenced
this issue
Mar 27, 2021
This commit is an improvement on Linux if you don't have the Wingdings font (which is provavly all Linux users by default). Without the Wingdings font, you get random characters that don't communicate the purpose of the checkbox widgets at all. With this commit, the north, south, east and west arrows render reasonably and the other five characters render as "missing character squares" which in my opinion makes it a little more obvious what was intended. Unfortunately, despite being only a few characters away from each other in unicode, on my system the cardinal directions are "normal width" unicode characters rendered at with a lighter weight, while the five missing characters are extra wide, and substantially heavier in their rendering. So they're definitely not "harmonious". If it works (and looks good) in Windows, I think this solution should be considered temporarily since it's an improvement on Linux, and it might get fixed before the commits #1 or #2, which is what should really be done long term. If this doesn't look good on Windows, we should revert to the original source code, and advise non-Windows users to somehow obtain and install the Wingdings font.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Opening and then canceling the "Map Settings" dialog causes an exception. The problem appears to be related to this issue.
On my system, the erroneous section contains:
Removing the problematic section from the config in ./bin/Debug/Trizbort.exe.config solves the problem, so this is somehow a "build" problem. It could be worked around in a binary Linux distribution by hand-editing the config post-build, but obviously, a better solution would be to not add the section in the first place.
Here's the rather large exception:
The text was updated successfully, but these errors were encountered: