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

hybrid char on Windows is x instead of × #301

Closed
mfrasca opened this issue Dec 24, 2017 · 4 comments
Closed

hybrid char on Windows is x instead of × #301

mfrasca opened this issue Dec 24, 2017 · 4 comments
Labels
need-stuff question if unsure, use this label Windows

Comments

@mfrasca
Copy link
Member

mfrasca commented Dec 24, 2017

@RoDuth ­— can you have a look at this, is this still the case?

    if sys.platform == 'win32':
        # TODO: for some reason using the cross as the hybrid
        # character doesn't work on windows
        Species.hybrid_char = 'x'
@mfrasca mfrasca added need-stuff Windows question if unsure, use this label labels Dec 24, 2017
@RoDuth
Copy link
Contributor

RoDuth commented Dec 30, 2017

Never noticed this but then don't have a lot of hybrids of this nature in the records (and our label reports output a multiplication sign for the hybrid flag).
From a VERY quick look I don't think its needed. I deleted these lines in a Win7 VM and ran a py2exe build (py2exe has some caused me some weird issues with unicode chars in the past) and all seemed to work fine.
I see this was committed in 2009 so it may be relevant for earlier version of windows (Win7 is the earliest version of windows I have easy access to at the moment) but considering that the multiplications character is used in other areas of ghini (notably when adding a species hybrid with the '*' trick) and hasn't yet lead to anyone saying they are having issues I'd say it should be fine to delete these lines.

@mfrasca
Copy link
Member Author

mfrasca commented Dec 30, 2017

please confirm that »all seemed to work fine« means that you:

  • deleted those two lines overriding '×' with 'x'
  • reinstalled the software
  • created a hybrid species
  • perform a search that contains the hybrid species in the results
  • see the '×' character in the results view
  • select it in the results
  • see the '×' character in the infobox — right hand side.
    if you can confirm this, just say "yes, confirmed" 😉

if it only means that you think the above would be the case, please say "no, not confirmed"
thank you so much, and have a nice change of year.

@RoDuth
Copy link
Contributor

RoDuth commented Dec 31, 2017

That was what was meant by

I deleted these lines in a Win7 VM and ran a py2exe build ... and all seemed to work fine.

What I didn't check is any other versions of windows its fair to assume that anything past 7 should be fine but possibly not XP or even Vista.

@mfrasca mfrasca added this to the PaloSeco-2017-12 milestone Jan 1, 2018
@mfrasca
Copy link
Member Author

mfrasca commented Jan 2, 2018

right, I'm still not sure I understand you're speaking of 'compiling' or of 'running'.
I'll remove the ×x translation and let users complain if things don't show correctly.

@mfrasca mfrasca closed this as completed in 30a43e0 Jan 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need-stuff question if unsure, use this label Windows
Projects
None yet
Development

No branches or pull requests

2 participants