Warlander / DeedPlanner-3

Village design and preview tool for Wurm Online and Unlimited
Apache License 2.0
24 stars 9 forks source link

Target Height input field enhancement #42

Closed KSunyo closed 1 year ago

KSunyo commented 1 year ago

Currently, the Target height input field works in the following way:

https://github.com/Warlander/DeedPlanner-3/assets/18518416/6cc8b346-8f4b-4750-b2ea-efdc33c1f124

As seen in the video, it's easy to get that zero unexpectedly added to your preferred height, causing users possibly applying unwanted heights to tiles.

I propose the following solution to the problem:

https://github.com/Warlander/DeedPlanner-3/assets/18518416/9caf426d-1d0f-44e0-b01b-43e674cb028f

The zero is still the default value for the input field, however, it is now a placeholder text that disappears the moment the user writes a number in the field. This should then also be the inital case for the field, as you can see here:

DP_TargetHeight_initial

Warlander commented 1 year ago

Sounds like a very good idea! I will implement this change in a free moment, it shouldn't take long.

Warlander commented 1 year ago

Implemented in development version, will be part of 3.2.0 release.

KSunyo commented 1 year ago

Wow, you were very fast, I can't wait the update! 💯