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 736052 - Avoid some deprecated properties in GtkBuilder files
Avoid some deprecated properties in GtkBuilder files
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-09-04 13:05 UTC by David King
Modified: 2015-01-25 13:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
avoid use-stock (9.01 KB, patch)
2014-09-04 13:06 UTC, David King
needs-work Details | Review
avoid has-resize-grip (1.37 KB, patch)
2014-09-04 13:06 UTC, David King
committed Details | Review
avoid use-stock (without new _Help string) (8.25 KB, patch)
2014-09-04 13:15 UTC, David King
committed Details | Review
avoid last use-stock use, and add _Help string (1.39 KB, patch)
2014-09-04 13:16 UTC, David King
accepted-commit_after_freeze Details | Review
add _Help string and mark it as translatable (1.41 KB, patch)
2014-09-04 15:01 UTC, David King
committed Details | Review
mark already-merged label strings as translatable (5.93 KB, patch)
2014-09-04 15:02 UTC, David King
committed Details | Review

Description David King 2014-09-04 13:05:13 UTC
There are some deprecated properties used in GtkBuilder files, which show up in the journal now:

(gnome-control-center:17428): GLib-GObject-WARNING **: The property GtkButton:use-stock is deprecated and shouldn't be used…

I also found that has-resize-grip was also still present.
Comment 1 David King 2014-09-04 13:06:27 UTC
Created attachment 285366 [details] [review]
avoid use-stock
Comment 2 David King 2014-09-04 13:06:56 UTC
Created attachment 285367 [details] [review]
avoid has-resize-grip
Comment 3 Bastien Nocera 2014-09-04 13:09:15 UTC
Review of attachment 285366 [details] [review]:

There's one new string: _Help

Can you split this up into something we can merge now, and the other small fix for 3.16?
Comment 4 Bastien Nocera 2014-09-04 13:10:11 UTC
Review of attachment 285367 [details] [review]:

Looks good. Add the "panels: " prefix to the commit subject
Comment 5 David King 2014-09-04 13:11:58 UTC
Review of attachment 285367 [details] [review]:

Pushed with that change, thanks!
Comment 6 David King 2014-09-04 13:15:43 UTC
Created attachment 285369 [details] [review]
avoid use-stock (without new _Help string)
Comment 7 David King 2014-09-04 13:16:28 UTC
Created attachment 285370 [details] [review]
avoid last use-stock use, and add _Help string
Comment 8 Bastien Nocera 2014-09-04 13:23:38 UTC
Review of attachment 285369 [details] [review]:

Looks good.
Comment 9 Bastien Nocera 2014-09-04 13:24:31 UTC
Review of attachment 285370 [details] [review]:

For 3.16, or before if you have the time to request that from gnome-i18n.

I wouldn't bother, it's just a warning that should be disabled in stable versions.
Comment 10 David King 2014-09-04 13:25:37 UTC
Review of attachment 285369 [details] [review]:

Thanks. Pushed to master.
Comment 11 David King 2014-09-04 15:01:48 UTC
Created attachment 285390 [details] [review]
add _Help string and mark it as translatable

Forgot to mark the new string as translatable.
Comment 12 David King 2014-09-04 15:02:55 UTC
Created attachment 285391 [details] [review]
mark already-merged label strings as translatable
Comment 13 Bastien Nocera 2014-09-04 15:13:32 UTC
Review of attachment 285391 [details] [review]:

Yes!
Comment 14 David King 2014-09-04 15:17:54 UTC
Review of attachment 285391 [details] [review]:

Pushed! Sorry for the trouble.
Comment 15 Bastien Nocera 2014-09-04 15:30:32 UTC
Review of attachment 285390 [details] [review]:

Ready for when we've branched
Comment 16 André Klapper 2015-01-24 08:58:12 UTC
(In reply to comment #15)
> Review of attachment 285390 [details] [review]:
> Ready for when we've branched

ping - Patch here is set to "accepted-commit_after_freeze" and might be nice to get it in before it is again blocke by a freeze.