bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

[Shingo Tanaka]

(set-face-attribute 'default nil :font "Noto Sans Mono CJK JP") doesn't take
effect on new frames on Android, even though the 2nd argument FRAME is nil.

Here is how to reproduce.
Maybe the font doesn't need to be the one I specified here.

0. Download NotoSansMonoCJKjp-VF.ttf from the url below and put it in ~/fonts 
1. Run Emacs w/o init file.
2. Evaluate (set-face-attribute 'default nil :font "Noto Sans Mono CJK JP")
   on *scratch* buffer.
3. You can see the displayed font is updated to Noto font.
4. Type C-x 5 2 (make-frame-command)
5. You can see the font is not Noto font but the initial font on the new frame.

Device: Xiaomi Pad 5
MIUI Version: MIUI Pad Global 14.0.5  14.0.5.0(TKXMIXM)
Android Version: 13 TKQ1.221013.002
Emacs Binary: emacs-30.0.50-29-arm64-v8a.apk
https://sourceforge.net/projects/android-ports-for-gnu-emacs/files/
Font: ~/fonts/NotoSansMonoCJKjp-VF.ttf
https://github.com/notofonts/noto-cjk
Sans/Variable/TTF/Mono/NotoSansMonoCJKjp-VF.ttf

(emacs-version)
"GNU Emacs 30.0.50 (build 1, aarch64-unknown-linux-android29)
 of 2023-09-11"

If any further information is needed, please let me know.

Regards,
Shingo

bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

[Eli Zaretskii]

> Date: Mon, 11 Sep 2023 21:10:37 +0900
> From: Shingo Tanaka <shingo.fg8@gmail.com>
> 
> (set-face-attribute 'default nil :font "Noto Sans Mono CJK JP") doesn't take
> effect on new frames on Android, even though the 2nd argument FRAME is nil.
> 
> Here is how to reproduce.
> Maybe the font doesn't need to be the one I specified here.
> 
> 0. Download NotoSansMonoCJKjp-VF.ttf from the url below and put it in ~/fonts 
> 1. Run Emacs w/o init file.
> 2. Evaluate (set-face-attribute 'default nil :font "Noto Sans Mono CJK JP")
>    on *scratch* buffer.
> 3. You can see the displayed font is updated to Noto font.
> 4. Type C-x 5 2 (make-frame-command)
> 5. You can see the font is not Noto font but the initial font on the new frame.

I must say that I'm uncomfortable with this use of script-specific
fonts.  These fonts are clearly intended to be used for specific
scripts/languages, not as the default font.  Emacs allows, via the
fontset feature, to configure a set of fonts that is optimized for
covering large portions of the Unicode codespace, for exactly this
reason: it is impractical to expect a single font have good support
both for, say, the Japanese script and for other scripts, including
advanced text-shaping features such as ligation etc.

So I really suggest not to use these Noto fonts as the default font,
but instead configure your fontset such that only the Japanese
characters are displayed using this font.

bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

[Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Shingo Tanaka <shingo.fg8@gmail.com> writes:

> (set-face-attribute 'default nil :font "Noto Sans Mono CJK JP") doesn't take
> effect on new frames on Android, even though the 2nd argument FRAME is nil.
>
> Here is how to reproduce.
> Maybe the font doesn't need to be the one I specified here.
>
> 0. Download NotoSansMonoCJKjp-VF.ttf from the url below and put it in ~/fonts 
> 1. Run Emacs w/o init file.
> 2. Evaluate (set-face-attribute 'default nil :font "Noto Sans Mono CJK JP")
>    on *scratch* buffer.
> 3. You can see the displayed font is updated to Noto font.
> 4. Type C-x 5 2 (make-frame-command)
> 5. You can see the font is not Noto font but the initial font on the new frame.
>
> Device: Xiaomi Pad 5
> MIUI Version: MIUI Pad Global 14.0.5  14.0.5.0(TKXMIXM)
> Android Version: 13 TKQ1.221013.002
> Emacs Binary: emacs-30.0.50-29-arm64-v8a.apk
> https://sourceforge.net/projects/android-ports-for-gnu-emacs/files/
> Font: ~/fonts/NotoSansMonoCJKjp-VF.ttf
> https://github.com/notofonts/noto-cjk
> Sans/Variable/TTF/Mono/NotoSansMonoCJKjp-VF.ttf
>
> (emacs-version)
> "GNU Emacs 30.0.50 (build 1, aarch64-unknown-linux-android29)
>  of 2023-09-11"
>
> If any further information is needed, please let me know.
>
> Regards,
> Shingo

Thanks, this should be fixed now.

bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

[Shingo Tanaka]

On Tue, 12 Sep 2023 11:39:05 +0900,
Po Lu wrote:
> 
> Thanks, this should be fixed now.

Thank you, I confirmed it's fixed.

On Mon, 11 Sep 2023 22:38:33 +0900,
Eli Zaretskii wrote:
> 
> So I really suggest not to use these Noto fonts as the default font,
> but instead configure your fontset such that only the Japanese
> characters are displayed using this font.

Thank you for the advice. I will consider to update my configuration accordingly.

Regards,
Shingo

bug#65865: 30.0.50; set-face-attribute doesn't take effect on new frames on Android

[Po Lu via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Shingo Tanaka <shingo.fg8@gmail.com> writes:

> On Tue, 12 Sep 2023 11:39:05 +0900,
> Po Lu wrote:
>> 
>> Thanks, this should be fixed now.
>
> Thank you, I confirmed it's fixed.

Closing.
Thanks for testing.