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

Danish language.. to the release branch this time.. #8237

Merged
merged 13 commits into from
Jun 25, 2014
Merged

Danish language.. to the release branch this time.. #8237

merged 13 commits into from
Jun 25, 2014

Conversation

bodhiBit
Copy link
Contributor

As requested, I'm submitting my pull request to the release branch instead of master..

I will see if I can get some other danish guys (and gals) to review it..

@bodhiBit
Copy link
Contributor Author

I've posted a link to this on a Danish news-site and requested reviews: http://newz.dk/forum/tagwall/review-af-dansk-oversaettelse-af-brackets-134128

So far I've gotten one, but fairly thorough review.. I will try to get as many improvements implemented as possible before tomorrow..

@ingorichter
Copy link
Contributor

@bodhiBit That sounds great. If you can get this done by tomorrow, it will make it into the 0.41 release.

<p>
Som tilføjelse til alle de gode sager der er indbygget i Brackets, har vores store og voksende samfund af
udviklere skabt over hundrede udvidelser, som tilføjer nyttig funktionalitet. Hvis der er
noget du har brug for, som Brackets ikke tilbyder, er det ret sandsynigt at nogen har lavet en udvidelse til
Copy link
Contributor

Choose a reason for hiding this comment

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

sandsynigt? Did you mean: sandsynligt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops.. missed that one.. fixed.. :)

@redmunds
Copy link
Contributor

@bodhiBit Thanks for your quick responses! I looked at the strings in google translate (http://translate.google.com) because we needed to know today if this was close enough to consider for Release 0.41 -- looks good to me. I will not merge it until tomorrow morning (my time) in case a native Danish speaker can give a more in-depth review and you want to make any more changes.

// Relative to the samples folder
"GETTING_STARTED" : "da/Kom godt i gang",
"ADOBE_THIRD_PARTY" : "http://www.adobe.com/go/thirdparty_da/",
"WEB_PLATFORM_DOCS_LICENSE" : "http://creativecommons.org/licenses/by/3.0/deed.da"
Copy link
Contributor

Choose a reason for hiding this comment

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

As @sbruchmann said, http://www.adobe.com/go/thirdparty_da/ redirects to http://www.adobe.com/dk/products/eula/third_party/ which is incorrect. This is because the Adobe EULA web site does not seem to have a translation for Danish, so remove the ADOBE_THIRD_PARTY line so that it defaults to root (English) translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like http://creativecommons.org/licenses/by/3.0/deed.da is also not translated, so remove that line as well.

define({
// Relative to the samples folder
"GETTING_STARTED" : "da/Kom godt i gang",
"WEB_PLATFORM_DOCS_LICENSE" : "http://creativecommons.org/licenses/by/3.0/deed.da"
Copy link
Contributor

Choose a reason for hiding this comment

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

I commented on the other thread right before your last update, so I'll post again to make sure you see this:

It looks like http://creativecommons.org/licenses/by/3.0/deed.da is also not translated, so remove that line as well.

@redmunds
Copy link
Contributor

I am done with my review. I'll wait until tomorrow morning to merge in case any other suggestions come in.

@bodhiBit
Copy link
Contributor Author

I don't know if this have been merged yet or you are waiting for word from me..

The translations is pretty much done.. I've implemented any corrections I've received and proofread it multiple times.. It's ready to go..

@redmunds
Copy link
Contributor

Thanks. Merging.

redmunds added a commit that referenced this pull request Jun 25, 2014
Danish language.. to the release branch this time..
@redmunds redmunds merged commit c0a8cc2 into adobe:release Jun 25, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants