Add context for "Automatic" strings
ClosedPublic
Actions

Authored by yurchor on May 2 2020, 11:05 AM.

Details

Reviewers

jgrulich
davidedmundson
ngraham
ecuadra

Group Reviewers

Plasma

Commits

R116:2bf19bb7c59d: Add context for "Automatic" strings

Summary

"Some of these topics are masculine gender and others are feminine gender, as "automatic" should be. The result is that I can't translate "automatic" correctly in Spanish (and other languages)." -- Eloy Cuadra, Spanish translation coordinator

Test Plan

Should work as is.

Diff Detail

Repository

R116 Plasma Network Management Applet

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

yurchor created this revision.May 2 2020, 11:05 AM

Restricted Application added a project: Plasma. · View Herald TranscriptMay 2 2020, 11:05 AM

Restricted Application added a reviewer: jgrulich. · View Herald Transcript

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

yurchor requested review of this revision.May 2 2020, 11:05 AM

Harbormaster completed remote builds in B26229: Diff 81736.May 2 2020, 11:05 AM

davidedmundson accepted this revision.May 2 2020, 2:57 PM

ngraham accepted this revision.May 2 2020, 10:37 PM

I think the right attribute to adding context to .ui files is "comment" instead of "context". The attribute "context" is only for .rc and .kcfg files ;-)
See https://api.kde.org/frameworks/ki18n/html/prg_guide.html#nocpp_ctxt

This revision now requires changes to proceed.May 3 2020, 12:01 AM

Use "comment" instead of "context", fix extracomment according to

https://tsdgeos.blogspot.com/2011/05/qt-designer-texts-disambiguation-vs.html

Harbormaster completed remote builds in B26253: Diff 81774.May 3 2020, 6:03 AM

@ecuadra Ping?

Yes. Sorry for the delay.

This revision was not accepted when it landed; it landed in state Needs Review.May 9 2020, 2:41 PM

Closed by commit R116:2bf19bb7c59d: Add context for "Automatic" strings (authored by yurchor). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			libs/editor/settings/ui/802-1x.ui (2 lines)
M			libs/editor/settings/ui/connectionwidget.ui (2 lines)
M			libs/editor/settings/ui/infiniband.ui (2 lines)
M			libs/editor/settings/ui/ipv4.ui (4 lines)
M			libs/editor/settings/ui/ipv6.ui (4 lines)
M			libs/editor/settings/ui/wificonnectionwidget.ui (4 lines)
M			libs/editor/settings/ui/wiredconnectionwidget.ui (4 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	81736	be3eaee		May 2 2020, 11:05 AM	★	★
Diff 2	81774	be3eaee	Use "comment" instead of "context", fix extracomment according to	May 3 2020, 6:03 AM	★	★
Diff 3	82379	7c91563	R116:2bf19bb7c59dd42321bd4fa8fc4eb62f888704c8	May 9 2020, 2:41 PM	★	★