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

Greek font looks bad/rendering issue #507

Open
stimfrog opened this issue Dec 17, 2024 · 13 comments
Open

Greek font looks bad/rendering issue #507

stimfrog opened this issue Dec 17, 2024 · 13 comments

Comments

@stimfrog
Copy link

Screenshot 2024-12-17 at 21-59-15 (1) Duolicious

@kero3009destiny
Copy link

Inter or Roboto can be a good fallback for Greek, pretty close to Montserrat.

@duogenesis duogenesis changed the title Greek font looks like shit/ rendering issue Greek font looks bad/rendering issue Jan 9, 2025
@DeepeshKalura
Copy link
Contributor

@duogenesis assigned this task to me.

We have to do simple task.

  1. I will download inter and roboto first. then put in the assets folder.
  2. There must a global theme file I will put that as a fallback where I found the greek.
  3. I think it just need to do in one are and valla the issue is resolve I think.

@duogenesis
Copy link
Member

duogenesis commented Jan 9, 2025

@DeepeshKalura Fonts go in the assets/ directory. Fonts are loaded in App.tsx. You can replace Montserrat with Roboto entirely, if Roboto's license permits us to distribute the font with Duolicious.

@DeepeshKalura
Copy link
Contributor

@duogenesis
Yeah thanks man but i predicted that. Since, you are using Montserrat, which is a Google Font, and Roboto is also a Google Font, I assume there shouldn't be any issues with the licensing.

However, I recommend verifying this information here: https://fonts.google.com/specimen/Roboto better then sorry

@duogenesis
Copy link
Member

@DeepeshKalura License looks fine to me

@DeepeshKalura
Copy link
Contributor

@DeepeshKalura Fonts go in the assets/ directory. Fonts are loaded in App.tsx. You can replace Montserrat with Roboto entirely, if Roboto's license permits us to distribute the font with Duolicious.

Hey man replace Montserrat with roboto entirely mean?

I was thinking to add new fonts roboto loadFonts and then in default text component ( might is the component ) make a roboto default and fallback montserrat.

I don't why we are removing Montserrat. I don't know where this image from by stimfrog from the codebase help me in that also

@duogenesis
Copy link
Member

Hey man replace Montserrat with roboto entirely mean?

Yes

I was thinking to add new fonts roboto loadFonts and then in default text component ( might is the component ) make a roboto default and fallback montserrat.

That sounds right, except for the fallback

I don't why we are removing Montserrat

If we use Roboto, it'll do everything Montserrat does but better. So Montserrat won't be needed

I don't know where this image from by stimfrog from the codebase

It's not in the code base. It looks like he wrote some Greek text in the 'About' section of his Profile. By the way, I forgot to mention you'll need to update the font in components/default-text-input.tsx.

@DeepeshKalura
Copy link
Contributor

@duogenesis Ok, so I am mostly correct for this time I will not be removing anything you will tag me in the code review stuff. Then I will start removing it if everything seems fine to you.

Rather I will create entirely new things separately and you check first tell me how you check also I think I will do it within 1 hour.

so I also change the component/default-text-input.tsx change ok

@duogenesis
Copy link
Member

@DeepeshKalura Are you going to fork duolicious/duolicious-frontend and then raise a pull request which contains your changes?

@DeepeshKalura
Copy link
Contributor

@duogenesis i already did and did some changes and reading codebase. LOL it first time seeing react-native code by the I am do mostly flutter for frontend

DeepeshKalura added a commit to DeepeshKalura/duolicious-frontend that referenced this issue Jan 9, 2025
Fixes duolicious#507

by adding robot to app.tsx and then chaning the default-text.tsx and default-text-input.tsx
DeepeshKalura added a commit to DeepeshKalura/duolicious-frontend that referenced this issue Jan 14, 2025
Fixes duolicious#507

missing static and resolve the error for roboto fonts
@duogenesis
Copy link
Member

I've looked into this some more. Can't find any fonts I like which support a wide variety of commonly used languages. For example, I found some fonts which support Greek and English, but then they don't support Japanese.

Fallback fonts aren't an option as React Native doesn't support them, and probably won't any time soon because of how infrequent the requests have been over the past few years (facebook/react-native#48625, facebook/react-native#32396, facebook/react-native#25798). Crappy fallback support can be hacked together by using logic to detect languages and dynamically changing the font, though that approach will break if someone uses two languages in the same bit of text.

If Facebook ever implements font fallbacks, I might replace Montserrat with Geologica, Fira Sans, or Commissioner to cover English and Greek. Then I'll find a fallback for the other languages weebs use.

@kero3009destiny
Copy link

If that is so, keeping the font as is or switching to one that looks similar but has at least one more supported script may be the safer option. I think duo will eventually need a full re-design at which point fonts can be reconsidered. I "professionally" do UI design and UX research and definitely want to discover and explore what may align with "what duolicious is" without being too corny or overwhelming.

@duogenesis
Copy link
Member

It would've been nice to have a professional to design the UI/UX before I wrote a line of code. Although I didn't have a good sense of "what Duolicious is" when I first made it, and the app's home page was very generic a couple of years ago. But now it's grown into the "chronically online" niche. So a UI that resonates most with the typical Discord user might be appropriate in the future.

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

No branches or pull requests

4 participants