DraqueT / PolyGlot

PolyGlot is a conlang construction toolkit.
MIT License
389 stars 44 forks source link

[Enhancement] move flag to use classic or simple conjugation autogeneration to language properties #1154

Open pe1uca opened 2 years ago

pe1uca commented 2 years ago

Description of the problem

These calls core.getPropertiesManager().setUseSimplifiedConjugations() are currently transparent to the user.
This means that if I have a part of speech that I want to set up with the simple form it's not completely clear that I'll be changing that set up for all parts of speech.

When you already have rules for an existing part of speech there's a proper warning that pops up, but if you are working with a new one, this message is not shown.
After you set up that new one with the simple view, you go to another one where you had multiple rules and find out that you lost your rules.

Proposed solution

Include this property as a checkbox in the language properties screen. Possibly also create a check and pop up to let the user know if there's any autogeneration with more than one rule and this will affect all of them.

DraqueT commented 2 years ago

This is a great catch. Thanks for both the attention to detail here and for the suggested solution!

I'm going to have to think on exactly how I'm ging to make this work technically, but I might look into making the complex/simplified system work on a per- PoS basis. If nothing else, I will absolutely expand the nature of the warning, as it's doesn't currently cover enough territory.