amethyst icon indicating copy to clipboard operation
amethyst copied to clipboard

[BUG] Allow to translate "mute" word

Open anthony-robin opened this issue 1 year ago • 15 comments

Describe the bug The "mute" word in the home screen popup filter menu is not translatable

To Reproduce Steps to reproduce the behavior:

  1. Go to main screen
  2. Click on "All follows"
  3. See "mute" word

Expected behavior A way to translate this word in crowdin

IMG_20230903_184451

anthony-robin avatar Sep 03 '23 16:09 anthony-robin

I've also seen this. I believe it's the name of the list and not a word defined in amethyst.

davotoula avatar Sep 06 '23 15:09 davotoula

Please @anthony-robin and @davotoula do you still see this?

KotlinGeekDev avatar Sep 27 '23 19:09 KotlinGeekDev

Please @anthony-robin and @davotoula do you still see this?

Hi @KotlinGeekDev, yes I still see it in the latest v0.78.2-FDROID version :)

anthony-robin avatar Sep 27 '23 19:09 anthony-robin

I still see the "mute" word in my list. I don't believe it's a bug, it's the actual name of the list. You cannot translate that just like you cannot translate a hashtag?

A thing has changed though. I used to be able to see all muted threads when selecting this list. Now it's blank (I guess spam filter blocks it).

It was a nice feature to view muted posts for anything interesting.

Screenshot_20230927-211443.png

davotoula avatar Sep 27 '23 19:09 davotoula

@davotoula I think a fix can be making the word a string resource, so that it can be translated.

KotlinGeekDev avatar Sep 27 '23 19:09 KotlinGeekDev

But isn't the word coming from the relay actually?

We don't want different language clients have different names for that list?

davotoula avatar Sep 27 '23 19:09 davotoula

Go for it! 😂

At the same time, can you look into why posts in that list are hidden now? They used to be visible!

davotoula avatar Sep 27 '23 19:09 davotoula

@davotoula you're right. It seems to come from the relay.

KotlinGeekDev avatar Sep 27 '23 19:09 KotlinGeekDev

Would it be possible to treat the mute list in a special way... And show a translated word in the client rather than just displaying what is returned from relay?

It could work if "mute" word is a standard in nostr... Otherwise it feels a bit unclean?

davotoula avatar Sep 27 '23 19:09 davotoula

That's the thought I had as well.

KotlinGeekDev avatar Sep 27 '23 19:09 KotlinGeekDev

A bit like a Decorator.. Pass most list names through from the relay but capture "mute" and display it translated?

davotoula avatar Sep 27 '23 20:09 davotoula

In a sense, yes.

KotlinGeekDev avatar Sep 27 '23 20:09 KotlinGeekDev

A bit like a Decorator.. Pass most list names through from the relay but capture "mute" and display it translated?

I actually found another way to do it: translate the word when it is being provided from the ViewModel(the view model that actually fetches the list).

KotlinGeekDev avatar Sep 27 '23 20:09 KotlinGeekDev

The issue here is that many clients use a "d=mute" list from NIP-51's 30000 event kind, instead of the mute list from 10000, which has no name.

We can probably migrate to event kind 10000 moving forward and stop using the mute list. That will allow the app to have a mute entry and translate it via crowdin.

vitorpamplona avatar Sep 27 '23 21:09 vitorpamplona

The issue here is that many clients use a "d=mute" list from NIP-51's 30000 event kind, instead of the mute list from 10000, which has no name.

We can probably migrate to event kind 10000 moving forward and stop using the mute list. That will allow the app to have a mute entry and translate it via crowdin.

I see. Makes sense.

KotlinGeekDev avatar Sep 27 '23 21:09 KotlinGeekDev

Please @anthony-robin I dont know if this is still an issue for you. @vitorpamplona maybe this can be closed already?(assuming clients have migrated to kind 10000 mute lists)

KotlinGeekDev avatar Aug 02 '24 14:08 KotlinGeekDev

Let's close the issue. Thank you for working on this :)

anthony-robin avatar Aug 02 '24 20:08 anthony-robin