Skip to content

Make Cyrillic diacritics display correctly. #4010

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

Closed
jcoyne opened this issue Feb 22, 2024 · 7 comments · Fixed by sul-dlss/searchworks_traject_indexer#1375 or sul-dlss/searchworks_traject_indexer#1583

Comments

@jcoyne
Copy link
Contributor

jcoyne commented Feb 22, 2024

The font of the "e" and "iaa" (with the diacritics) in the 710 looks too big in the SearchWorks display.

Message sent from: https://searchworks.stanford.edu/view/413028

From Vitus:

Not exactly precomposed letter with diacritic vs. two separate characters but in this case two ways to encode the ligature. MARC21 uses to map the ligature as two Unicode characters (U+FE20 and U+FE21) but in 2004 the mapping was changed to one character (U+0361). Symphony followed that, so all ligatures in Symphony were U+0361. But OCLC still to this day uses U+FE20 and U+FE21. I have argued with them for years about that. If you compare the word Strategii︠a︡ in https://searchworks.stanford.edu/view/in00000002927 with Strategii͡a in https://searchworks.stanford.edu/view/4152692 you will see this difference.
What's interesting is that in the Folio instance record of in00000002927, the two halves look just fine, but in SearchWorks it appears bolded.

@dnoneill dnoneill self-assigned this Feb 23, 2024
@dnoneill
Copy link
Contributor

@saseestone Is going to see if someone can edit the record on folio-test. I believe this might be a data issue that we don't necessarily want to fix. The way that the record displays in folio is shown below. Based on the view in folio it is showing up correctly (just with more spacing) in searchworks.

Screenshot 2024-02-23 at 2 52 10 PM

@dnoneill
Copy link
Contributor

Alisa updated the the title for https://searchworks-stage.stanford.edu/view/in00000002927 on folio-test from Strategii︠a︡ to Strategii͡a. It is displaying correctly in searchworks. I think the only fix for this would be possibly change the font but from what I am understand we are using Stanford standards so we probably won't want do this. I am moving this into the backlog.

@saseestone
Copy link

saseestone commented Feb 29, 2024

@dnoneill I'm actually hoping that we can do something in SW (maybe in the indexer?) that would correct this. Does the FOLIO update that Alissa did yesterday give us a clue about something we could do to render the correct ligature? (i.e. U+0361)?

Chatting with @cbeer just now, it seems like maybe there are some things we could try?

I also chatted with @ahafele yesterday, wondering if there was a chance that this would be fixed in FOLIO. She wrote "The data on our side will remain the same until OCLC changes their policy on this character..." I guess it's possible down the road that FOLIO could do what Symphony used to do, but it feels like we should continue to pursue managing this in the indexer or display until then. That's my $.02 worth. :)

@jcoyne
Copy link
Contributor Author

jcoyne commented Apr 4, 2025

This was reverted in sul-dlss/searchworks_traject_indexer#1415

@jcoyne
Copy link
Contributor Author

jcoyne commented Apr 4, 2025

But was it not fixed again by sul-dlss/searchworks_traject_indexer#1418 ?

@jcoyne
Copy link
Contributor Author

jcoyne commented Apr 4, 2025

And then disabled in sul-dlss/searchworks_traject_indexer#1419

@cbeer
Copy link
Member

cbeer commented Apr 8, 2025

The previous Cyrillic fix throws tons of honeybadger errors:
sul-dlss/searchworks_traject_indexer#1589

@jcoyne jcoyne self-assigned this Apr 8, 2025
@jcoyne jcoyne assigned jcoyne and unassigned jcoyne Apr 24, 2025
@taylor-steve taylor-steve self-assigned this Apr 29, 2025
@cbeer cbeer changed the title Placeholder: Cyrillic diacritics follow-up Make Cyrillic diacritics display correctly. Apr 29, 2025
@taylor-steve taylor-steve removed their assignment Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants