Skip to content

Conversation

@megaocta
Copy link
Contributor

@megaocta megaocta commented Mar 4, 2019

Fixed some rough translations

Minimize to tray=Minimieren zum Tray
Name=Name
No host name specified=keinen Hostnamen angegeben
No host name specified=Keinen Hostnamen angegeben
Copy link

Choose a reason for hiding this comment

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

Why not "Kein Hostname angegeben". In which context does this get displayed?

Copy link
Contributor

Choose a reason for hiding this comment

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

function TConnOptionsForm.Validate: boolean;
begin
Result:=False;
edHost.Text:=Trim(edHost.Text);
if Trim(edHost.Text) = '' then begin
Page.ActivePage:=tabConnection;
edHost.SetFocus;
MessageDlg(sNoHost, mtError, [mbOK], 0);
exit;
end;
edProxy.Text:=Trim(edProxy.Text);
if tabProxy.TabVisible and cbUseProxy.Checked and (edProxy.Text = '') then begin
Page.ActivePage:=tabProxy;
edProxy.SetFocus;
MessageDlg(sNoProxy, mtError, [mbOK], 0);
exit;
end;
Result:=True;
end;

Copy link
Member

Choose a reason for hiding this comment

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

@leonsoft-kras I thought you replied by mistake here? Seems unrelated 😅

Copy link
Member

@PeterDaveHello PeterDaveHello left a comment

Choose a reason for hiding this comment

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

Hi @megaocta, thanks for your contribution, do you think you can find someone good at German can help review this change? Sorry that I don't speak German so I can't properly review it, translators are not reliable in this case.

Copy link
Member

@PeterDaveHello PeterDaveHello left a comment

Choose a reason for hiding this comment

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

Checked by GPT-4

@PeterDaveHello PeterDaveHello merged commit 85189b1 into transmission-remote-gui:master May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants