630167 - update Polish L10n files for DOMi Bug 615156

Status: RESOLVED FIXED

(Mozilla Localizations :: pl / Polish, defect)

Description

[Adrian Kalla [:adriank]]

The Polish localization files for the DOM Inspector have to be updated and "pl"
should be added back to "ALL_LOCALES" because of DOMi Bug 615156.

Comment 1

[Adrian Kalla [:adriank]]

Attachment: patch v1

Comment 2

[Colby Russell :crussell (no longer Mozilla-ing)]

Comment on attachment 508375 [details] [diff] [review]
patch v1

>+<!ENTITY screenXCoord.label "ekran: x">
>+<!ENTITY screenYCoord.label "ekran: y">

Is this best?  I don't speak Polish, but if there's a more natural way to communicate these that allows you to drop the colons (which suggest to me that the choice+word order might be a little "forced"), go with it instead.

Comment 3

[Adrian Kalla [:adriank]]

(In reply to comment #2)
> Comment on attachment 508375 [details] [diff] [review]
> patch v1
> 
> >+<!ENTITY screenXCoord.label "ekran: x">
> >+<!ENTITY screenYCoord.label "ekran: y">
> 
> Is this best?  I don't speak Polish, but if there's a more natural way to
> communicate these that allows you to drop the colons (which suggest to me that
> the choice+word order might be a little "forced"), go with it instead.

Yes, it's clearly not the best... The best would probably be:
+<!ENTITY screenXCoord.label "współrzędna pozioma ekranu">
+<!ENTITY screenYCoord.label "współrzędna pionowa ekranu">
But I'm not sure, if there is enough space for that...?

Comment 4

[Marek Stępień [:marcoos, inactive]]

Yeah, "ekran: x" sounds weird and I wouldn't probably understand it. At least, not instantly.

Version from comment 3 seems OK, but shouldn't you update xCoord.label and yCoord.label to match, i.e. drop the "x" and "y" completely and use "współrzędna pozioma/pionowa" in all of these entities?

Comment 5

[Adrian Kalla [:adriank]]

(In reply to comment #4)
> Yeah, "ekran: x" sounds weird and I wouldn't probably understand it. At least,
> not instantly.
> 
> Version from comment 3 seems OK, but shouldn't you update xCoord.label and
> yCoord.label to match, i.e. drop the "x" and "y" completely and use
> "współrzędna pozioma/pionowa" in all of these entities?

Hmmm, makes sense. I'll try first, if this fits in the available space.

Comment 6

[Adrian Kalla [:adriank]]

Attachment: patch v2

patch includes the feedback from the comments

Comment 7

[Adrian Kalla [:adriank]]

pushed: https://hg.mozilla.org/dom-inspector/rev/6f8f919b58cb
thanks