Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Update strings.js Italian Language #8061

Merged
merged 3 commits into from
Jun 10, 2014
Merged

Update strings.js Italian Language #8061

merged 3 commits into from
Jun 10, 2014

Conversation

Denisov21
Copy link
Contributor

Correct small errors!

Denisov21 added 2 commits June 7, 2014 14:21
@@ -197,9 +197,9 @@ define({
*/
"STATUSBAR_CURSOR_POSITION" : "Linea {0}, Colonna {1}",
"STATUSBAR_SELECTION_CH_SINGULAR" : " \u2014 Selezionata {0} colonna",
"STATUSBAR_SELECTION_CH_PLURAL" : " \u2014 Selezionata {0} colonne",
"STATUSBAR_SELECTION_CH_PLURAL" : " \u2014 Selezionate {0} colonne",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Denisov21 The message (in English) is intended to reflect the fact that {0} columns have been selected. Does your translation change the verb tense to "select", rather than "selected"?

@bchintx bchintx self-assigned this Jun 10, 2014
Fixed any issue with Bryan Chin
@Denisov21
Copy link
Contributor Author

@bchintx Correct, thank you!

@bchintx
Copy link
Contributor

bchintx commented Jun 10, 2014

@Denisov21 Sorry for the confusion, but, actually, I'm questioning whether your proposed change is correct. I'll have to admit that I don't know Italian, but just a quick Google translation suggests that the previous translation better maps to the original English message. In other words, that string is displayed in the Status bar, showing how many columns lines are currently selected after a Find command is run.

Assuming that you're more fluent in Italian than I am, can you please confirm that the change is better than the original?

Sorry for the trouble.

@Denisov21
Copy link
Contributor Author

@bchintx I speak Italian for a long time, but I think the best translation is still the second (78e1ff7)! If you agree to restore everything as before and can you merge?

@bchintx
Copy link
Contributor

bchintx commented Jun 10, 2014

@Denisov21 Perfect -- just wanted to confirm that. I'll merge your changes now. Thanks!

bchintx added a commit that referenced this pull request Jun 10, 2014
Update strings.js Italian Language
@bchintx bchintx merged commit 21ad6f0 into adobe:master Jun 10, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants