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

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 26229
Build 26247: arc lint + arc unit

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 (2 lines)
M			libs/editor/settings/ui/ipv6.ui (2 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	★	★