-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Translate in Italian for Brackets 1.4 #11318
Conversation
Translate in Italian for Brackets 1.4
|
@nicolo-ribaudo Would you mind doing a quick review for these changes? |
src/nls/it/strings.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better Le preferenze del progetto non sono in un file JSON valido. Il file verrà aperto ... (Similar to line 90)
src/nls/it/strings.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trueper visualizzare le anteprime delle immaginianche se non hanno un'estensione
|
Done |
|
@nicolo-ribaudo I started review the comments :) |
|
I've translated some JSLint-related strings differently from |
|
good translation for me! |
|
Good for me! |
|
@Denisov21 I've corrected these errors: Denisov21#1 |
|
I think @nicolo-ribaudo can close this PR becouse @marcelgerber merge this PR (#11117). |
|
That PR was about other strings |
|
@Denisov21 @nicolo-ribaudo @fredev @marcelgerber |
|
No, because @Denisov21 hasn't merged Denisov21/brackets#1yet. |
|
Scusa @nicolo-ribaudo che cosa significa "No, because @Denisov21 hasn't merged Denisov21/brackets#1yet ? Io non posso unire nulla! |
|
@Denisov21 You should merge Denisov21#1 . Here is a guide on how to merge a pr: https://help.github.com/articles/merging-a-pull-request/ Italian translation: |
Italian translation review
|
@abose @marcelgerber Now this pr can be merged |
|
There should be a comma at the end of line 650 |
Fix JSHint errors in nls/it/strings.js
|
@nicolo-ribaudo I've done! I merge with success. Thank @nicolo-ribaudo very much! |
Translate in Italian for Brackets 1.4
|
@Denisov21 @nicolo-ribaudo @fredev Thanks everyone for the quick changes. |
Translate in Italian for Brackets 1.4