Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Strings missing for translation on edit list screenΒ #362

Closed

Description

@gunchleoc

Describe the bug
When editing a list in current unstable, some strings are still in English

To Reproduce
Steps to reproduce the behavior:

  1. Use a locale not English
  2. Click on 'Lists'
  3. Click on an existing, empty list
  4. The contents of "Mode" and "No entries found" are in English

Screenshots
cawbird-edit-empty-list

Activity

  1. gunchleoc commented on Apr 9, 2021

    @gunchleoc
    Author

    P.S: Spelling right-click menus (language / Spelling Suggestions) also have untranslatable English strings in them.

  2. IBBoard commented on Apr 10, 2021

    @IBBoard
    Owner

    Looks like Private/Public is going to be tricky because Baedert used it as both the value and the string. It can be done, but we still need to pass "private" or "public" to Twitter.

    "No entries found" was trying to be translatable, but it got missed out of the "translate these files" list.

  3. added a commit that references this issue on Apr 10, 2021
  4. IBBoard commented on Apr 10, 2021

    @IBBoard
    Owner

    Can I have a screenshot of the spelling suggestions? I think it might be out of our control. I can't find "Spelling" anywhere in the code base, and it's almost certainly added by gspell automatically.

  5. added a commit that references this issue on Apr 10, 2021
  6. gunchleoc commented on Apr 10, 2021

    @gunchleoc
    Author

    I can't get a screenshot because it is blocking the print screen button. To reproduce, compose a new tweet, type some nonsense and right-click on it.

    gspell is only 76 words, so I've done the translation and the next Gnome update should fix it.

    Looks like Private/Public is going to be tricky because Baedert used it as both the value and the string. It can be done, but we still need to pass "private" or "public" to Twitter.

    Let's put this on low priority.

  7. IBBoard commented on Apr 10, 2021

    @IBBoard
    Owner

    I can't get a screenshot because it is blocking the print screen button. To reproduce, compose a new tweet, type some nonsense and right-click on it.

    That's inconvenient! I tend to use Glimpse in those situations and do a time-delayed screenshot. (Other image editors and/or screenshot tools may be available πŸ˜‰)

    gspell is only 76 words, so I've done the translation and the next Gnome update should fix it.

    Just to confirm, is "Spelling Suggestions…" in there? If so then I'll close this until we find bugs.

    Let's put this on low priority.

    It wasn't too bad in the end. I'll update Unstable so that you can test 03210f7.

  8. gunchleoc commented on Apr 12, 2021

    @gunchleoc
    Author

    Dug, I could have taken a time delayed screenshot with GIMP. "Spelling Suggestions…" is part of the gspell package, so let's consider this one solved.

    "No entries found" is still in English.

  9. gunchleoc commented on Apr 12, 2021

    @gunchleoc
    Author

    I have found a final untranslatable string, Custom in the translation engine list. Settings -> Tweets

    cawbird-custom-se

  10. added a commit that references this issue on Apr 12, 2021
  11. IBBoard commented on Apr 12, 2021

    @IBBoard
    Owner

    "No entries found" is working for me. Maybe I didn't get the patch sorted properly.

    I've just added "Custom" (and the other sites, just in case) as translatable. Thanks for being so thorough with the checks!

    screenshot

    Builds are in progress. Give it five minutes to finish, and then maybe another five to fifteen to "publish" it to the repo and it should be good. Anything after cawbird_1.3.2-1+94.1 should be updated.

  12. gunchleoc commented on Apr 13, 2021

    @gunchleoc
    Author

    Your screenshot for "No entries found" is of the search function, and for slightly different strings too (No user/tweet found). I was on the list screen, after creating a new, empty list and then selecting that list. The string is on Transifex and I translated it 3 days ago. It's still missing from unstable deb version 1.3.2-1+95.1.

  13. IBBoard commented on Apr 13, 2021

    @IBBoard
    Owner

    That's interesting, because they both use the ListBox object, so it should be the same message! The string isn't anywhere else in the source code. I'll have more of a look.

  14. gunchleoc commented on Apr 13, 2021

    @gunchleoc
    Author

    Yep, it's strange. Try calling reset_placeholder_text() to see whether that fixes it?

  15. IBBoard commented on Apr 13, 2021

    @IBBoard
    Owner

    SearchPage doesn't seem to call the reset, so it would be odd if that fixed it. Putting some debug in beside the two references to "No entries found" and it only ever shows that the translation returns the English. Even on the search page, which definitely shows Gaelic!

    [Edit] Wait a minute… the search page says "No tweets found" and "No users found". Which are set in its constructor. The list page is using the default "No entries found" string. Which is set differently.

    [Edit 2] Even if I reset during on_join (which is called every time you switch to a page) then it doesn't get the translated strings, even though SearchPage gets them right in its constructor. Clearly SOMETHING is behaving oddly.

  16. IBBoard commented on Apr 13, 2021

    @IBBoard
    Owner

    Or it's really trivial and when Gettext has a "fuzzy" string then it defaults to the original text (English in this case) rather than showing a potentially incorrect translation. #facepalm

    Screenshot from 2021-04-13 21-02-24

    Fixed by #373. New "unstable" builds have started.

  17. gunchleoc commented on Apr 13, 2021

    @gunchleoc
    Author

    Thanks for fixing! πŸŽ‰

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

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions