Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DejaVu has no Regular with Putty #1249

Closed
3 tasks done
Finii opened this issue May 19, 2023 · 4 comments · Fixed by #1255
Closed
3 tasks done

DejaVu has no Regular with Putty #1249

Finii opened this issue May 19, 2023 · 4 comments · Fixed by #1255

Comments

@Finii
Copy link
Collaborator

Finii commented May 19, 2023

🗹 Requirements

This issue was originally raised by liaoya (Huifeng Shen) on Gitter. Thank you.

  • I have searched the issues for my issue and found nothing related and/or helpful
  • I have searched the FAQ for help
  • I have searched the Wiki for help

🎯 Subject of the issue

Experienced behavior:
Putty shows only Oblique, Bold Oblique and Bold for DejaVuSansM Nerd Font Mono
Also Bold Oblique is depicted as Bold

Expected behavior:

Example symbols:

🔧 Your Setup

  • Which font are you using (e.g. Anonymice Powerline Nerd Font Complete.ttf)?
    • Please give the full filename DejaVuSansM Nerd Font Mono
  • Where did you get the file from (download link, self patched, source downloaded from link...)
    • www.nerdfonts.com
  • Which terminal emulator are you using (e.g. iterm2, urxvt, gnome, konsole)?
    • putty latest
  • Are you using OS X, Linux or Windows? And which specific version or distribution?
    • Win10

★ Screenshots (Optional)

grafik

@Finii
Copy link
Collaborator Author

Finii commented May 22, 2023

Hmm, did we not prevent this?

image

Edit: Ugg, that is the source font, not the patched font...

image

@Finii
Copy link
Collaborator Author

Finii commented May 22, 2023

@Finii
Copy link
Collaborator Author

Finii commented May 22, 2023

Font is patched with --has-no-italic, but fsSelect incorrect:

$ find . -name '*.ttf' -and -name '*Mono-*' -print -exec bash -c 'ttfdump {} | grep fsS' \;
./Regular/DejaVuSansMNerdFontMono-Regular.ttf
	 fsSelection:		 0x01c0 	 'Regular '
./Bold/DejaVuSansMNerdFontMono-Bold.ttf
	 fsSelection:		 0x01a0 	 'Bold '
./Italic/DejaVuSansMNerdFontMono-Oblique.ttf
	 fsSelection:		 0x0380 	 ''
./Bold-Italic/DejaVuSansMNerdFontMono-BoldOblique.ttf
	 fsSelection:		 0x03a0 	 'Bold '

Finii added a commit that referenced this issue May 22, 2023
[why]
For fonts that have no Italic but an Oblique - i.e. when Oblique shall
replace the Italic role in RIBBI font grouping (classic group of 4) -
that grouping fails.

This affects DejaVu on Putty.

[how]
For RIBBI grouping only the classic bits are considered. That means that
for fonts that have Oblique instead of Italic (and not additionally) we
need to set the ITALIC bit and the OBLIQUE bit. This has been
overlooked.

Cite from the specs:

> This bit, unlike the ITALIC bit (bit 0), is not related to style-linking
> in applications that assume a four-member font-family model comprised
> of regular, italic, bold and bold italic. It may be set or unset
> independently of the ITALIC bit. In most cases, if OBLIQUE is set, then
> ITALIC will also be set, though this is not required.

Fixes: #1249

Reported-by: Huifeng Shen <liaoya@gmail.com>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Finii added a commit that referenced this issue May 22, 2023
[why]
For fonts that have no Italic but an Oblique - i.e. when Oblique shall
replace the Italic role in RIBBI font grouping (classic group of 4) -
that grouping fails.

This affects DejaVu on Putty.

[how]
For RIBBI grouping only the classic bits are considered. That means that
for fonts that have Oblique instead of Italic (and not additionally) we
need to set the ITALIC bit and the OBLIQUE bit. This has been
overlooked.

Cite from the specs:

> This bit, unlike the ITALIC bit (bit 0), is not related to style-linking
> in applications that assume a four-member font-family model comprised
> of regular, italic, bold and bold italic. It may be set or unset
> independently of the ITALIC bit. In most cases, if OBLIQUE is set, then
> ITALIC will also be set, though this is not required.

[note]
Also increase font-patcher version.

Fixes: #1249

Reported-by: Huifeng Shen <liaoya@gmail.com>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Finii added a commit that referenced this issue May 22, 2023
[why]
For fonts that have no Italic but an Oblique - i.e. when Oblique shall
replace the Italic role in RIBBI font grouping (classic group of 4) -
that grouping fails.

This affects DejaVu on Putty.

[how]
For RIBBI grouping only the classic bits are considered. That means that
for fonts that have Oblique instead of Italic (and not additionally) we
need to set the ITALIC bit and the OBLIQUE bit. This has been
overlooked.

Cite from the specs:

> This bit, unlike the ITALIC bit (bit 0), is not related to style-linking
> in applications that assume a four-member font-family model comprised
> of regular, italic, bold and bold italic. It may be set or unset
> independently of the ITALIC bit. In most cases, if OBLIQUE is set, then
> ITALIC will also be set, though this is not required.

[note]
Also increase font-patcher version.

Fixes: #1249

Reported-by: Huifeng Shen <liaoya@gmail.com>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
LNKLEO pushed a commit to LNKLEO/Nerd that referenced this issue Nov 24, 2023
[why]
For fonts that have no Italic but an Oblique - i.e. when Oblique shall
replace the Italic role in RIBBI font grouping (classic group of 4) -
that grouping fails.

This affects DejaVu on Putty.

[how]
For RIBBI grouping only the classic bits are considered. That means that
for fonts that have Oblique instead of Italic (and not additionally) we
need to set the ITALIC bit and the OBLIQUE bit. This has been
overlooked.

Cite from the specs:

> This bit, unlike the ITALIC bit (bit 0), is not related to style-linking
> in applications that assume a four-member font-family model comprised
> of regular, italic, bold and bold italic. It may be set or unset
> independently of the ITALIC bit. In most cases, if OBLIQUE is set, then
> ITALIC will also be set, though this is not required.

[note]
Also increase font-patcher version.

Fixes: ryanoasis#1249

Reported-by: Huifeng Shen <liaoya@gmail.com>
Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant