Merge lp://staging/~alexander-wilms/beat-box/translation-de into lp://staging/beat-box

Proposed by 982c80311320c1b
Status: Merged
Merged at revision: 512
Proposed branch: lp://staging/~alexander-wilms/beat-box/translation-de
Merge into: lp://staging/beat-box
Diff against target: 1936 lines (+1150/-273)
14 files modified
po/beatbox.pot (+516/-77)
po/de.po (+520/-82)
src/Dialogs/AddPodcastWindow.vala (+5/-5)
src/Dialogs/PlaylistNameWindow.vala (+3/-3)
src/Dialogs/SetMusicFolderConfirmation.vala (+4/-4)
src/Dialogs/SmartPlaylistEditor.vala (+52/-52)
src/Dialogs/SongEditor.vala (+5/-5)
src/Dialogs/StationEditor.vala (+3/-3)
src/Dialogs/SyncWarningDialog.vala (+3/-3)
src/Dialogs/TransferFromDeviceDialog.vala (+8/-8)
src/Views/ListView/MusicTreeView.vala (+15/-15)
src/Views/ListView/RadioListView.vala (+1/-1)
src/Views/ViewWrapper.vala (+1/-1)
src/Widgets/SideTreeView.vala (+14/-14)
To merge this branch: bzr merge lp://staging/~alexander-wilms/beat-box/translation-de
Reviewer Review Type Date Requested Status
xapantu (community) Approve
Review via email: mp+91180@code.staging.launchpad.net

Description of the change

Added more translate strings, but I guess someone with more coding skills would need to take care of the playlist names and stuff containing variables.

To post a comment you must log in.
Revision history for this message
xapantu (xapantu) wrote :

Some minor things could be changed, e.g. in "<b>Text</b>", we don't want the <b> to be translatable, so, we prefer having this: "<b>%s</b>".print(_("Text")) instead of _("<b>Text</b>"). It is a bit more verbose, but it is easier for the translators. I will do it when I merge your branch :)

Thanks for this branch, really appreciated :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches