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.

Sign up for GitHub

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

Jump to bottom

SmallCaps only partially present in 4.0-beta8 #556

Closed
TKrepo opened this issue Apr 12, 2023 · 5 comments
Closed

SmallCaps only partially present in 4.0-beta8 #556

TKrepo opened this issue Apr 12, 2023 · 5 comments

Comments

@TKrepo
Copy link

TKrepo commented Apr 12, 2023 •
edited
Loading

Small Caps are nice feature to emphasize Text.

There are some Small Caps present in Inter 4.0-beta8, but only a few. Interestingly their availability depends on whether Capital or Lower Case letters are displayed as Small Caps. Some of the Stylistic Alternatives are present (Capital I), some are missing (Capital G).

Tested with the hinted TTF (Light) in LibreOffice 7.4.6 on Windows 10.

Capture-or8
@rsms
Copy link
Owner

rsms commented Apr 12, 2023 •
edited
Loading

This is intentional (of course.) The small-cap glyphs that are present are required or needed for different reasons (other than serving a smcp feature.) There’s a feature request here already for adding true small caps to Inter: #32 (won’t do this until after v4.0 is released.)

@rsms
Copy link
Owner

rsms commented Apr 12, 2023

Closing this as it’s not a bug and is already covered by #32 as a feature request

@rsms rsms closed this as not planned Won't fix, can't repro, duplicate, stale Apr 12, 2023
@tim-janik
Copy link

Closing this as it’s not a bug and is already covered by #32 as a feature request

Oh, I just came here to report a regression... So v3.10 is not actually supposed to support small-caps?
For the record, typesetting "Devices" in small-caps worked in 3.10 but not the newest v4 beta.

inter4smallcaps

@rsms
Copy link
Owner

rsms commented Jun 20, 2023

Closing this as it’s not a bug and is already covered by #32 as a feature request

Oh, I just came here to report a regression... So v3.10 is not actually supposed to support small-caps? For the record, typesetting "Devices" in small-caps worked in 3.10 but not the newest v4 beta.

inter4smallcaps

The second line is faux small-caps, generated by the software you’re using, from the regular caps. That’s why the stroke weight is too thin.

@tim-janik
Copy link

tim-janik commented Jun 20, 2023 •
edited
Loading

The second line is faux small-caps, generated by the software you’re using, from the regular caps. That’s why the stroke weight is too thin.

Ok, thanks for pointing that out. This is rendered with Chrome-114 on Linux btw.
Which leaves the question why 3.10 is rendered with faux small-caps when 4.0 isn't.
Just a guess, did 3.10 maybe not contain a c2sc setting?
If so, wouldn't it make sense to leave that out of the 4.0 builds until proper support is ready?

tim-janik added a commit to tim-janik/anklang that referenced this issue Jun 27, 2023
@tim-janik
UI: b/menutitle.js: use drop-caps while InterVariable-4.0 small-caps …
732ab47 
…are broken

See also: rsms/inter#556 (comment)

Signed-off-by: Tim Janik <[email protected]>
tim-janik added a commit to tim-janik/anklang that referenced this issue Jun 27, 2023
@tim-janik
Merge branch 'InterDisplay', based on InterVariable-4.0 font
89c9d01 
* InterDisplay:
  UI: dark.scss: add a fallback font stack
  UI: theme.scss: remove outdated font stuff
  UI: mixins.scss: move b-kbd-hotkey here
  UI: elements.scss: use b-font-weight-bold() and b-font-weight-bolder()
  UI: b/*.vue, b/*.js: use b-font-weight-bold() and its variants
  UI: dark.scss: define b-font-weight-{normal|bold|bolder} mixins
  UI: b/menutitle.js: use drop-caps while InterVariable-4.0 small-caps are broken
	See also: rsms/inter#556 (comment)
  UI: elements.scss: leave font handling to theme, globals and shadow
  UI: b/pianoroll.js: use b-canvas-font
  UI: b/partthumb.vue: use b-canvas-font
  UI: b/clipview.js: use b-canvas-font
  UI: shadow.scss: use b-ui-font() inside shadow DOMs
  UI: globals.scss: make use of b-ui-font()
  UI: dark.scss: configure b-ui-font() for "InterVariable" in InterDisplay style
  UI: globals.scss: leave font-face defs to index.html
  UI: index.html: cleanup preloads, add @font-face for "InterVariable"
  UI: b/icon.js: add small comment
  UI: Makefile.mk: download InterVariable.woff2
  UI: stylelintrc.cjs: disable "font-family-name-quotes" to allow quoted font names

Signed-off-by: Tim Janik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rsms @tim-janik @TKrepo