diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-10-08 22:23:47 +0200 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2018-10-09 21:00:44 -0700 |
commit | f40612d48d7c4ae33a99a10e0ac497b10f7b98aa (patch) | |
tree | 01affb1a81ad032fc9c7be69ae01760c170c5955 /CHANGELOG.md | |
parent | 7a1ceee4e791d8fcc44b571b7453d2ebb2308388 (diff) | |
download | subsurface-f40612d48d7c4ae33a99a10e0ac497b10f7b98aa.tar.gz |
Map: avoid ugly "invalid dive site" message in location box
Under certain conditions the user was presented an ugly
"invalid dive site" message. The condition would arise because
the proxy-model which selects the list of dive sites and the code
which creates a proposed dive site name had different filter
conditions:
- The proxy would select any dive site containing the text
- The name-proposing code searched for dive sites *starting*
with the text.
If the user entered a text contained by a dive site name, but
no dive site would start with the second line was filled with
a dummy text. This text would be kept if it contained the text
entered by the user.
To avoid this problem, if no dive site is found, use an empty
string instead. This will be filtered out by the proxy because
it does not contain the user-entered string.
Yes, that's horribly subtle, therefore add a comment. But ultimately,
this should be solved in a less brittle way.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'CHANGELOG.md')
0 files changed, 0 insertions, 0 deletions