Difference between revisions of "Sword25/TODO"

Jump to navigation Jump to search
163 bytes removed ,  15:22, 15 January 2011
Colour passing seems to be endian-aware, and swapping R and B is probably wrong because they seem to have the correct values already.
(Added another note about the menu text colour glitch.)
(Colour passing seems to be endian-aware, and swapping R and B is probably wrong because they seem to have the correct values already.)
Line 5: Line 5:
* Graphics:
* Graphics:
** Problems with start menu:
** Problems with start menu:
**# Button texts are the wrong colour. According to [http://www.mobygames.com/game/windows/broken-sword-25-the-return-of-the-templars/screenshots/gameShotId,330004/ this screenshot on MobyGames] the text should be yellow, not light blue. The text is set by CBitmapButton:OnCreate() in menu.lua, which calls Text::setColor() in gfx/text.cpp. This is then rendered by RenderedImage::blit() in gfx/image/renderedimage.cpp, where it uses the colour to tint the text data. I do get the correct colour (or close enough) if I swap cr and cb, and that might actually be the correct fix. But I don't know. Come to think of it, if it passes the colour as an array to setColor() and that function treats it like an integer, isn't that endian-unsafe?
**# Button texts are the wrong colour. According to [http://www.mobygames.com/game/windows/broken-sword-25-the-return-of-the-templars/screenshots/gameShotId,330004/ this screenshot on MobyGames] the text should be yellow, not light blue. The text is set by CBitmapButton:OnCreate() in menu.lua, which calls Text::setColor() in gfx/text.cpp. This is then rendered by RenderedImage::blit() in gfx/image/renderedimage.cpp, where it uses the colour to tint the text data. I do get the correct colour (or close enough) if I swap cr and cb, but that's probably not the correct fix.
** Problems with the exit menu:
** Problems with the exit menu:
**# Wrong shading of 'yes/no' glyphs
**# Wrong shading of 'yes/no' glyphs
415

edits

Navigation menu