Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Polish context menu - pixelorama.nsi #972

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

piotrtrkostrzewski94
Copy link

@piotrtrkostrzewski94 piotrtrkostrzewski94 commented Jan 10, 2024

I added a translation to the "Edit project with Pixelorama" context menu into Polish.

I added a translation to the Edit project with Pixelorama context menu into Polish.
@dphaldes
Copy link
Contributor

Translations are done through crowdin on https://crowdin.com/project/pixelorama. You can submit translations there and they get imported into the project every few weeks. It's easier for translators to work on it this way.

@piotrtrkostrzewski94
Copy link
Author

piotrtrkostrzewski94 commented Jan 10, 2024

I know this, I'm a Pixelorama translator there. There is no context menu added on CrowdIn. It's not a problem for me to add a commit here. I hope the creators don't mind this.

@dphaldes
Copy link
Contributor

Well in that case, that entry needs to be added to crowdin instead :)

@dphaldes
Copy link
Contributor

Never mind this is for installer and not the main application. I have no clue how those are handled 😅

@piotrtrkostrzewski94
Copy link
Author

Maybe they will add it for other languages, there is no need for Polish :) In my opinion, it is for the main application because it is in the context menu :)

@@ -81,7 +81,7 @@

; Multilingual support

!insertmacro MUI_LANGUAGE "English"
!insertmacro MUI_LANGUAGE "Polski"
Copy link
Member

Choose a reason for hiding this comment

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

This change looks incorrect to me, it seems to be replacing English with Polish rather than adding it as a new translation. I'm not entirely familiar with how .nsi files work so I'm not sure what the correct solution is here.

@@ -152,7 +152,7 @@
WriteRegStr HKCR "Pixelorama project\DefaultIcon" "" "$INSTDIR\pxo.ico"

WriteRegStr HKCR "Pixelorama project\shell\open\command" "" '$INSTDIR\${APPNAME}.exe "%1"'
WriteRegStr HKCR "Pixelorama project\shell\edit" "" "Edit project in ${APPNAME}"
Copy link
Member

Choose a reason for hiding this comment

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

Same as above.

Choose a reason for hiding this comment

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

In the file I sent, there is a Polish version of the context menu on line 155. Should I add a new file
Screenshot_2024-01-11-15-38-07-831_com android chrome
called pixelorama-pl.nsl?

Choose a reason for hiding this comment

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

I made a mistake, I could have added it immediately as a new file and added PR. Sorry, I'm still learning Github. I sent the correct (I hope) PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants