Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Thermometer's value is incorrectly drawn #85

Closed
mrfatguy opened this issue Mar 23, 2024 · 2 comments · Fixed by #150
Closed

Thermometer's value is incorrectly drawn #85

mrfatguy opened this issue Mar 23, 2024 · 2 comments · Fixed by #150
Assignees
Labels
bug Something isn't working

Comments

@mrfatguy
Copy link
Contributor

mrfatguy commented Mar 23, 2024

https://github.com/niner-games/lost-dutchman-mine/wiki/Interface.-Control-Panel

The bar that is suppose to denote current temperature is drawn to far to the left and "jumps" of the actual thermometer's bulb.

It is less noticeable in the default (windowed) app view:

t1

And much more when game is in full-screen mode:

t2

In addition:

  • temperature knob is using pale #ff0000 red color which makes it flat while original game is using two colors:
    • #ae0000 for the outset / border
    • #ef0000 for the inset,
  • the bulb image is missing
  • the background image is cut without surrounding 1-pixel border which also makes the whole thermometer flat.

Please, see the original image for reference:

image

Please, not that in-game temperature never drops below 30 F, so the bottom part of the thermometer (with bulb) can be drawn as static, all-the-time the same image:

image

More thermometer's images can be found at our Wiki.

@MechanikGamer
Copy link
Contributor

Fixed it here #149

@MadejaMaciej
Copy link
Contributor

We will need separate ticket for handling the temperature and height of the bar itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants