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

Click on individuals support #11

Closed
wants to merge 5 commits into from
Closed

Click on individuals support #11

wants to merge 5 commits into from

Conversation

06Games
Copy link
Contributor

@06Games 06Games commented Jan 2, 2021

This pull request allows you to navigate in the tree by clicking on individuals or to open its page if the individual is at the root of the tree

@magicsunday
Copy link
Owner

Thanks for your contribution.

For now I only picked the 820c1d3 commit as I'm unsure in which direction the development will lead and if clicking on the root person should lead to its individual page. Updating the chart while clicking on a person does not update the chart title, maybe this should collapse the subtree instead.

@06Games
Copy link
Contributor Author

06Games commented Jan 6, 2021

Regarding the orientation of development, I see two possibilities:

  • either stay on the view of a tree showing only the ancestry, and in this case, perhaps clicking on people should not change the root person but open the page of the clicked individual or as you said, show/hide the ancestors of the person.
  • if not, it could be to develop a real alternative to the default tree, by allowing to display the children as well as the wives. We could also think of adding an information banner when clicking on the person to have a quick overview of the information without having to open the page of the individual, a bit like what Geneanet offers (for example: https://gw.geneanet.org/evangalli_w?lang=en&n=dragon&oc=0&p=pierre+antoine&type=tree) In this case, I find that moving by clicking on individuals makes perfect sense and would be a real addition.

As for the title, I had to remove the name of the selected person from it in order to overcome the problem of updating. Although I don't find it disturbing, I understand that it can be inconvenient to lose this information from the title

@06Games
Copy link
Contributor Author

06Games commented Jan 6, 2021

Also, I would like to do another pull request to add a French translation but I did not fully understand how files with an mo extension open. I can open them with Notepad++ but the beginning of the file does not correspond to text. Should I use specific software or just edit the text without worrying about unrecognized characters?

@magicsunday
Copy link
Owner

magicsunday commented Jan 6, 2021

I use PoEdit to edit the language files. https://poedit.net/ You only edit the .po files. The .mo files are the created binary files.

@magicsunday
Copy link
Owner

I added you commits but did some small changes to 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.

None yet

2 participants