Skip to content

update Spanish translation - #1062

Merged
justinclift merged 1 commit into
sqlitebrowser:masterfrom
lulol:master
Jul 24, 2017
Merged

justinclift merged 1 commit into
sqlitebrowser:masterfrom
lulol:master

Conversation

@lulol

@lulol lulol commented Jul 23, 2017

Copy link
Copy Markdown
Contributor

update Spanish translation

@justinclift

Copy link
Copy Markdown
Member

Thanks @lulol, that was pretty quick. 😄

Should it be merged as is, or would it be better to make a bunch of screenshots showing the strings at runtime first?

@lulol

lulol commented Jul 23, 2017

Copy link
Copy Markdown
Contributor Author

I think that is not necessary to take screenshots. The translation was mostly straightforward.

The only thing that may worth to check is if the resulting text formatting looks right for the key cypher dialog and the Plot load button tooltip, since in both there were too long lines I did split them in the translation.

@justinclift

Copy link
Copy Markdown
Member

The Plot loading tooltip seems fine (bottom right corner):
plot_load_tooltip

The key cypher dialog I'll check later, when I switch to my OSX desktop later on tonight. 😄

@justinclift

Copy link
Copy Markdown
Member

They SQLCipher dialog seems ok too, so this does look good to merge as-is:
screen shot 2017-07-24 at 12 58 26

Thanks heaps @lulol. 😄

@justinclift
justinclift merged commit 7baea3b into sqlitebrowser:master Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants