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 781622 - gtkplacesview.ui: Update helper label to include IPv6 addresses
gtkplacesview.ui: Update helper label to include IPv6 addresses
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: .General
unspecified
Other All
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2017-04-22 19:16 UTC by Carlos Soriano
Modified: 2017-04-22 19:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gtkplacesview.ui: Update helper label to include IPv6 addresses (1.32 KB, patch)
2017-04-22 19:16 UTC, Carlos Soriano
none Details | Review
gtkplacesview.ui: Remove translatable from server label helper (1.21 KB, patch)
2017-04-22 19:41 UTC, Carlos Soriano
none Details | Review
gtkplacesview.ui: Update helper label to include IPv6 addresses (1.27 KB, patch)
2017-04-22 19:41 UTC, Carlos Soriano
committed Details | Review
gtkplacesview.ui: Remove translatable from server label helper (1.22 KB, patch)
2017-04-22 19:54 UTC, Carlos Soriano
committed Details | Review

Description Carlos Soriano 2017-04-22 19:16:24 UTC
.
Comment 1 Carlos Soriano 2017-04-22 19:16:29 UTC
Created attachment 350249 [details] [review]
gtkplacesview.ui: Update helper label to include IPv6 addresses

This is a backport from 16bce171683d58. Thing is Nautilus pulled the
code from master until now, so right now gtk-3-22 and Nautilus 3.24
differ.

Would be great to have them share the same code, and at the same time
have this small string helper issue fixed.
Comment 2 Carlos Soriano 2017-04-22 19:17:08 UTC
Forgot to mention, this is for 3.22
Comment 3 Matthias Clasen 2017-04-22 19:22:39 UTC
Review of attachment 350249 [details] [review]:

tbh, marking this string as translatable seems entirely pointless
Comment 4 Carlos Soriano 2017-04-22 19:41:35 UTC
Created attachment 350250 [details] [review]
gtkplacesview.ui: Remove translatable from server label helper

Translating it seems pointless if we can use a non-translatable example
such as gnome.org instead of foo.example.com.

This will help to make changes in here without breaking string freeze.
Comment 5 Carlos Soriano 2017-04-22 19:41:42 UTC
Created attachment 350251 [details] [review]
gtkplacesview.ui: Update helper label to include IPv6 addresses

This is a backport from 16bce171683d58. Thing is Nautilus pulled the
code from master until now, so right now gtk-3-22 and Nautilus 3.24
differ.

Would be great to have them share the same code, and at the same time
have this small string helper issue fixed.
Comment 6 Matthias Clasen 2017-04-22 19:43:11 UTC
Review of attachment 350250 [details] [review]:

I agree
Comment 7 Matthias Clasen 2017-04-22 19:43:50 UTC
Review of attachment 350251 [details] [review]:

ok
Comment 8 Carlos Soriano 2017-04-22 19:54:48 UTC
Created attachment 350252 [details] [review]
gtkplacesview.ui: Remove translatable from server label helper

Translating it seems pointless if we can use a non-translatable example
such as gnome.org instead of foo.example.com.

This will help to make changes in here without breaking string freeze.
Comment 9 Carlos Soriano 2017-04-22 19:55:27 UTC
Attachment 350252 [details] pushed as 72d1353 - gtkplacesview.ui: Remove translatable from server label helper