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

Importing the localized fitting file(xml) exported from EVE #2653

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

jeffy-g
Copy link
Contributor

@jeffy-g jeffy-g commented Nov 30, 2024

Hello!

I haven't played EVE for a while, but I've started playing again recently.

Inevitably, I ended up using Pyfa again,
but I feel that I need to make some changes to the import of localized fitting files, so I am submitting a PR.

For simple testing, I created localized fitting files for each language and placed them in tests/localized-fitting-files.
You can see how it works by importing all the xml files at once with Pyfa.

It also seems that the character limit for fitting notes has been increased from 400 to 500,
so I've included those changes in this PR.

If the character count exceeds 500, the text will turn red.

thanks for reading o7

Fitting description limit extended to 500 characters

localized fitting file sample
if use `applicationbound()` instead of `threadbound()`.
@jeffy-g jeffy-g force-pushed the fix-localized_import/note_limit branch from e5b3e5b to f9f0806 Compare December 1, 2024 04:32
@jeffy-g jeffy-g force-pushed the fix-localized_import/note_limit branch from 29e0a54 to ff1bd81 Compare December 1, 2024 07:40
@jeffy-g
Copy link
Contributor Author

jeffy-g commented Dec 2, 2024

The German and Russian localized fitting files export xml files with the exact same contents regardless of important names.

hint attr is <official name>, text content is <official name>*

The layout of the wx.ProgressDialog is messed up... :‑(

@DarkFenX
Copy link
Member

DarkFenX commented Dec 5, 2024

I will take a look at it (and possibly merge) later. Will make today release w/o this, sorry

@jeffy-g
Copy link
Contributor Author

jeffy-g commented Dec 6, 2024

Okay! np :-)

Please check it out, if there's anything you don't like or any problems, please fix it.

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.

2 participants