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

Updated Russian locale for Release 0.41#8366

Closed
MaximKhlobystov wants to merge 2 commits intoadobe:releasefrom
MaximKhlobystov:release
Closed

Updated Russian locale for Release 0.41#8366
MaximKhlobystov wants to merge 2 commits intoadobe:releasefrom
MaximKhlobystov:release

Conversation

@MaximKhlobystov
Copy link
Contributor

Little grammar mistakes were edited. Some text was paraphrased. Some missing fields were added.

… was paraphrased. Some missing fields were added.
@busykai busykai self-assigned this Jul 12, 2014
@busykai
Copy link
Contributor

busykai commented Jul 12, 2014

General comment: please replace all the tabs with 4 spaces.

Copy link
Contributor

Choose a reason for hiding this comment

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

Since you're already doing this, could you please change this entry from FILE_ALREADY_EXISTS to ENTRY_WITH_THE_SAME_NAME_EXISTS and change the translation as well. Original entry is:

    "ENTRY_WITH_SAME_NAME_EXISTS"       : "A file or directory with the name <span class='dialog-filename'>{0}</span> already exists.",

@busykai
Copy link
Contributor

busykai commented Jul 12, 2014

@gthacoder, thanks a lot! You need to submit this to the master branch: release 41 is out and the release branch for 42 hasn't been created yet. I seems like we forgot to delete the release branch (@redmunds, did we?).

This looks good -- nice corrections, it was missing proof-reading! I only have these minor comments + a general comment on use of tabs (use 4 spaces instead).

There are some strings that are not being used by Brackets any longer and needs to be cleaned up. If you could do it, that'd be awesome. If not, I'll clean them up once this merge.

@MaximKhlobystov
Copy link
Contributor Author

@busykai, thanks a lot for your feedback. I didn't see the tabs until now. Now it's all fixed. I will resubmit it (to the master branch).

@redmunds
Copy link
Contributor

@busykai

seems like we forgot to delete the release branch (@redmunds, did we?).

We haven't deleted the release branch because we'll just be creating another one for the next Release.

@busykai
Copy link
Contributor

busykai commented Jul 14, 2014

Closing this in favor of #8371.

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.

3 participants