Opened 15 years ago

Closed 15 years ago

Last modified 8 months ago

#8331 closed patch (wontfix)

Updated internal font

Reported by: SF/damienguard Owned by: fingolfin
Priority: normal Component: GUI
Keywords: Cc:
Game:

Description

Hi there, first patch so go easy on me ;-)

Moved the internal font definition out to it's own
conditionally included file and made the following changes;

1. Lots of symbols and accented chars added in the
style of the font where they were missing (adopted
codepage 850 as thats what existing symbols mapped to)

2. Extended to the full 255 character table (previously
a char over 236 in scummvm.ini would behave strangely)

3. Removed unused data (saved a few bytes)

4. Major source tidy (much easier to see what is
spacing, where the chars are etc).

5. Char bitmap data now hex so it can be more easily
edited by hand (wish C supported binary consts)

6. Changed a couple of symbols slightly to better match
rest of font (tilde and beta)

There are still some gaps with foreign chars - notably
upper-case ones where there is an accent because the
font style doesn't fit too well. I will try and
address these in a later patch when I am happier with them.

Also note that the // comments in the source next to
each char in the 127+ range will only display properly
if your viewer/editor's font is codepage 850.

Ticket imported from: #900575. Ticket imported from: patches/436.

Attachments (1)

newgui.diff.zip (4.5 KB) - added by SF/damienguard 15 years ago.
Font file and updated newgui.cpp

Download all attachments as: .zip

Change History (5)

Changed 15 years ago by SF/damienguard

Attachment: newgui.diff.zip added

Font file and updated newgui.cpp

comment:1 Changed 15 years ago by fingolfin

Owner: set to fingolfin
Resolution: wontfix

comment:2 Changed 15 years ago by fingolfin

I fail to see the point in this patch. We already *have* in CVS a full
fledged font rendere, with exchangable fonts. It's currently disabled, but
once 0.6.0 is out of the door we will resume discussing it and use it to
replace the old font rendering code.

Is there *anything* this offers over our already existing new font code?
(None of the things you list above do, maybe there is something I am
missing, though?)

comment:3 Changed 15 years ago by fingolfin

Status: newclosed

comment:4 Changed 8 months ago by digitall

Component: GUI
Note: See TracTickets for help on using tickets.