update Polish L10n files for DOMi Bug 615156

RESOLVED FIXED

Status

Mozilla Localizations
pl / Polish
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: adriank, Assigned: adriank)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

7 years ago
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.
(Assignee)

Comment 1

7 years ago
Created attachment 508375 [details] [diff] [review]
patch v1
Attachment #508375 - Flags: review?(Sevenspade)
Attachment #508375 - Flags: feedback?(marcoos+bmo)
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.
(Assignee)

Comment 3

7 years ago
(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...?
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?
(Assignee)

Comment 5

7 years ago
(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.
(Assignee)

Comment 6

7 years ago
Created attachment 509167 [details] [diff] [review]
patch v2

patch includes the feedback from the comments
Attachment #508375 - Attachment is obsolete: true
Attachment #509167 - Flags: review?(Sevenspade)
Attachment #508375 - Flags: review?(Sevenspade)
Attachment #508375 - Flags: feedback?(marcoos+bmo)
(Assignee)

Updated

7 years ago
Attachment #509167 - Flags: review?(marcoos+bmo)
Attachment #509167 - Flags: review?(marcoos+bmo) → review+
Attachment #509167 - Flags: review?(Sevenspade) → review+
(Assignee)

Comment 7

7 years ago
pushed: https://hg.mozilla.org/dom-inspector/rev/6f8f919b58cb
thanks
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.