anaseto / boohu

Break Out Of Hareka's Underground, a roguelike game.
https://codeberg.org/anaseto/boohu
ISC License
128 stars 8 forks source link

Colours are messed up #25

Closed VedVid closed 5 years ago

VedVid commented 5 years ago

I'm playing standalone ASCII version on Windows 10. Release 0.10 worked well, but playing 0.11 is nearly impossible due to colour scheme. Walls are "covered" by FOV layer:
image

Solarized palette is not good either, because it's eye-tiring:
image

On screenshots I'm using cmder terminal emulator, but on default windows command line effect is the same.

Screenshot of Boohu 0.10:
image

anaseto commented 5 years ago

Does the problem persist if you toggle twice again the dark/light setting with = and then b? (just to be sure it's not because of some incompatibility of config between versions) I may have screwed something up, some of the drawing code was factorized during development of the last version. I'll have a look into it.

VedVid commented 5 years ago

It helped - after toglling dark/light settings, walls are displayed correctly even in default mode.

edit: I'm using different versions of Boohu in separate folder, so local config files should not overlap.

anaseto commented 5 years ago

There was indeed a bug when no config file was present (it did kind of a mix of the two dark/light themes). The above commit should fix the issue.

VedVid commented 5 years ago

Interesting... I build ASCII release from master by go get -u git.tuxfamily.org/boohu/boohu.git then go build.

Now, the default mode is this one:
image
but if I toggle FOV mode and quit game, then start it again, whole screen is, well, bright:
image In that case, toggling FOV once results in regular light mode, and next switch to the default one.

VedVid commented 5 years ago

Ups, it's even more complicated. Let me post gif here:
2018-11-30 17h55_36

anaseto commented 5 years ago

Oh, yes, hm, I've found the main culprit: I changed the way dark/light worked (now there is indeed a dark with green LOS, and a bright one), but the toggling for the windows 8-colour version only got half updated. I'll fix this asap.

anaseto commented 5 years ago

Hopefully now fixed with the last commit. Now there should be for the 8-colour palette a dark or light background, with green light in both cases.

Thanks for pointing out this issue!

VedVid commented 5 years ago

No problem, it's in my best interests as well ;)

Unfortunately, it still looks strange. I can switch between
image
and
image

This light blue is nice hue, but I don't think it's intended...

anaseto commented 5 years ago

Oops! Should be fixed now. The thing is, on my current OpenBSD machine I have to use the tcell backend and cannot even test myself the termbox backend I use for Windows, Linux and OS X binaries, portability is hard :-)

VedVid commented 5 years ago

In that case, it's good that I'm on Windows and have plenty of time. And Go environment set up already :)

Now dark FOV looks as it's supposed to look, but the light one is still problematic - whole screen has bright background, only foreground changes. And tiles in player's FOV are still green.
image

anaseto commented 5 years ago

The light one did work in fact as I intented after the theme change I did after people pointed out to me that the contrast between LOS background and the rest was too much, and the results looks ok with 16 colors.

But the everything bright one is indeed not very pretty with 8 colours, so I switch it back again to how it worked before for the 8-color version! :-)

VedVid commented 5 years ago

Thanks! :) I think we can close this issue now.

anaseto commented 5 years ago

Great! And thanks again for helping debugging this :)

VedVid commented 5 years ago

I'm glad I was able to help :)