After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 780460 - Untranslatable strings
Untranslatable strings
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: I18N
3.24.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-03-23 16:58 UTC by Piotr Drąg
Modified: 2017-03-23 20:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Mark Bookmark Properties string for translation (1.49 KB, patch)
2017-03-23 17:08 UTC, Michael Catanzaro
committed Details | Review
Attempt to fix translation of Favorites string (1.83 KB, patch)
2017-03-23 17:14 UTC, Michael Catanzaro
none Details | Review
Fix untranslated strings in clear data dialog (1.13 KB, patch)
2017-03-23 19:11 UTC, Michael Catanzaro
committed Details | Review
Fix untranslated Favorites string in bookmarks popover title (1.19 KB, patch)
2017-03-23 19:32 UTC, Michael Catanzaro
committed Details | Review

Description Piotr Drąg 2017-03-23 16:58:58 UTC
“Bookmark Properties” in src/bookmarks/ephy-bookmark-row.c is not marked for translation.

Also clicking the bookmarks button on the header bar → Tags → Favorites shows “Favorites” header in English.
Comment 1 Michael Catanzaro 2017-03-23 17:08:55 UTC
The following fix has been pushed:
2ad3d63 Mark Bookmark Properties string for translation
Comment 2 Michael Catanzaro 2017-03-23 17:08:58 UTC
Created attachment 348588 [details] [review]
Mark Bookmark Properties string for translation

Oops
Comment 3 Piotr Drąg 2017-03-23 17:13:20 UTC
Strings from src/clear-data-dialog.c are also in English.
Comment 4 Michael Catanzaro 2017-03-23 17:14:45 UTC
Created attachment 348589 [details] [review]
Attempt to fix translation of Favorites string

This is untested. Piotr, do you have time to try testing this?
Comment 5 Piotr Drąg 2017-03-23 17:25:47 UTC
I can’t, sorry!
Comment 6 Piotr Drąg 2017-03-23 17:47:47 UTC
Another case is strings in lib/ephy-form-auth-data.c, visible in Seahorse.
Comment 7 Michael Catanzaro 2017-03-23 19:08:26 UTC
(In reply to Piotr Drąg from comment #6)
> Another case is strings in lib/ephy-form-auth-data.c, visible in Seahorse.

Which strings specifically? I don't see anything there than can be translated. E.g. we cannot translate schema keys.
Comment 8 Michael Catanzaro 2017-03-23 19:11:33 UTC
The following fix has been pushed:
99827b5 Fix untranslated strings in clear data dialog
Comment 9 Michael Catanzaro 2017-03-23 19:11:36 UTC
Created attachment 348600 [details] [review]
Fix untranslated strings in clear data dialog
Comment 10 Piotr Drąg 2017-03-23 19:14:53 UTC
(In reply to Michael Catanzaro from comment #7)
> (In reply to Piotr Drąg from comment #6)
> > Another case is strings in lib/ephy-form-auth-data.c, visible in Seahorse.
> 
> Which strings specifically? I don't see anything there than can be
> translated. E.g. we cannot translate schema keys.


“Password for %s in a form in %s” and “Password in a form in %s”. Similar strings from GOA (<https://git.gnome.org/browse/gnome-online-accounts/tree/src/goabackend/goautils.c#n508>) are shown translated in Seahorse.
Comment 11 Michael Catanzaro 2017-03-23 19:32:55 UTC
The following fix has been pushed:
71568c3 Fix untranslated Favorites string in bookmarks popover title
Comment 12 Michael Catanzaro 2017-03-23 19:32:58 UTC
Created attachment 348602 [details] [review]
Fix untranslated Favorites string in bookmarks popover title

We translated it in most places, but missed a spot.
Comment 13 Michael Catanzaro 2017-03-23 19:33:52 UTC
(In reply to Piotr Drąg from comment #10)
> “Password for %s in a form in %s” and “Password in a form in %s”. Similar
> strings from GOA
> (<https://git.gnome.org/browse/gnome-online-accounts/tree/src/goabackend/
> goautils.c#n508>) are shown translated in Seahorse.

OK but these are both actually marked for translation, and ephy-form-auth-data.c is present in POTFILES.in.
Comment 14 Piotr Drąg 2017-03-23 19:38:29 UTC
(In reply to Michael Catanzaro from comment #13)
> OK but these are both actually marked for translation, and
> ephy-form-auth-data.c is present in POTFILES.in.

Right, so it’s different. Would you like me to file a new bug for it?
Comment 15 Michael Catanzaro 2017-03-23 19:46:00 UTC
(In reply to Piotr Drąg from comment #14)
> (In reply to Michael Catanzaro from comment #13)
> > OK but these are both actually marked for translation, and
> > ephy-form-auth-data.c is present in POTFILES.in.
> 
> Right, so it’s different. Would you like me to file a new bug for it?

Yeah, thanks. But I have no clue what's going on, so somebody is going to have to debug it for it to be fixed. :(

But you got lots of other untranslatable strings fixed, so that's good!
Comment 16 Piotr Drąg 2017-03-23 20:09:01 UTC
(In reply to Michael Catanzaro from comment #15)
> Yeah, thanks. But I have no clue what's going on, so somebody is going to
> have to debug it for it to be fixed. :(
> 

Reported as bug #780476.

> But you got lots of other untranslatable strings fixed, so that's good!

*We* did that. :)