Skip to content

Conversation

@mickael-menu
Copy link
Member

What kind of change does this PR introduce? (Bug fix, feature, docs update, other)

Upgrade the embedded AccessibleDfa font files to fix an issue with some glyphs always displayed with a black color. I took this opportunity to add the italic and bold variants as well.

What is the current behaviour? (You can also link to an open issue here)

When switching on the night theme, the commas are disappearing (they stay black).

Screenshot 2023-02-02 at 11 13 53

Screenshot 2023-02-02 at 11 14 10

What is the new behaviour?

AccessibleDfa 1.7 fixes the issue.

Screenshot 2023-02-02 at 11 14 50

@danielweck
Copy link
Member

fixes #56
?

@mickael-menu
Copy link
Member Author

Does anyone have the permissions to merge this? cc @aferditamuriqi @danielweck

@aferditamuriqi
Copy link
Member

Does anyone have the permissions to merge this? cc @aferditamuriqi @danielweck

i don't , sorry

@aferditamuriqi
Copy link
Member

@mickael-menu looks like only Jiminy has write access, but we can ask @llemeurfr to add someone to that team https://github.com/orgs/readium/teams/readium-css-committers

@danielweck danielweck merged commit b1392b9 into readium:develop Mar 11, 2024
@danielweck
Copy link
Member

let's go! (I checked the PR contents but didn't actually test it)

@mickael-menu mickael-menu deleted the update-accessibledfa branch March 11, 2024 16:10
@mickael-menu
Copy link
Member Author

Thanks Daniel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants