[haiku-commits] Re: haiku: hrev52159 - src/servers/app/font docs/interface_guidelines src/kits/network/libnetapi src/servers/app

  • From: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 5 Aug 2018 08:25:19 +0200

On Sat, Aug 04, 2018 at 06:59:44PM -0500, looncraz wrote:

Not a fan: we already have a method for this (IsFullAndHalfFixed()). Why
introduce another?
Also, "monospaced" is a relatively known term, and full-half is more
exact and descriptive.


Full-half makes no sense in English, it's a contradiction in terms.

Where did you find full-half?

The function is Full*And*HalfFixed, which means "includes both
full-width and halfwidth characters". The name is what is used in
Unicode and in OpenType:

https://en.wikipedia.org/wiki/Halfwidth_and_fullwidth_forms

The proper term is Duospaced.

We would only be changing the name of IsFullAndHalfFixed().

We already have an API, and we can't change its name because we're
trying to stay somewhat compatible with BeOS here.

If you search for "full and half font" the results, in order:

You get better results if you search for "full and half *fixed* font".
Of course removing the main keyword from the search terms does not help.


I confirmed that BeOS does not consider full and half-width fonts as
monospace.

I would just merge the changes as they were proposed:
- Implement IsFullAndHalfFixed to return true only for
  fullwidth/halfwidth fonts,
- Leave IsFixed as it is (only for monospace fonts)
- Allow be_fixed_font to be either a monospace or fullwidth/halfwidth font,
- Let Appearance filter fonts using both of these flags

While adding IsMonospace / IsDuospace would be nice, I would say there
is too much overlap with the current APIs, so it will be a little
confusing for API users to pick the right function. I would rather keep
things simple.

Usually apps will just use be_fixed_font, anyways (and maybe adjust the
size), so they won't deal with the font query functions. It seems weird
to me that we have a setting for a fixed-width font in Appearance, but
the main app needing it (Terminal) has its own setting.

-- 
Adrien.

Other related posts: