nurpax / petmate

PETSCII editor with Electron/React/Redux
MIT License
181 stars 14 forks source link

Colorize empty cells? #84

Closed Esshahn closed 6 years ago

Esshahn commented 6 years ago

This is not a bug and I totally get it from the technical side. It just occurred to me that this might not be the desired behavior. Or maybe it is. Would like to get feedback from @nurpax and @Viza64

What happens:

It acts like color ram and screen ram actually. Please check out the video attached.

Question is: should $20 generally be treated as "transparent" and not accept color when there's nothing to see?

colorizing.mov.zip

nurpax commented 6 years ago

It’d be nice to make that assumption. It might be broken by custom charset loading. (space might not be at $20). Otoh charsets most likely alwwys have an empty char do could search for it from the font bitmap.

Esshahn commented 6 years ago

Good thinking with the problems that might arise later with custom charsets. And as mentioned already, it does behave totally correct. Marq's editor does not have this "issue", as he does not feature a "character only" draw mode. With what you just noted I think it might be a good idea to leave it as is currently. What do you think?

Esshahn commented 6 years ago

Closing this. It will come up again if this is a problem in the future, but let's keep the backlog clean.