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

add paste source citation icon to the INDI names #51

Closed
HerzScheisse opened this issue Jan 31, 2024 · 7 comments
Closed

add paste source citation icon to the INDI names #51

HerzScheisse opened this issue Jan 31, 2024 · 7 comments

Comments

@HerzScheisse
Copy link
Contributor

Hi,

thanks for this awesome module and all its great features.

is it possible to add the "paste source citation" icon to the names aswell?
repo_paste_source

thanks
/Holger

@Jefferson49
Copy link
Owner

Hello Holger,

thank you for reporting the issue.

For simplification, I wanted to use the copy/paste mechanism only for facts/events, because all facts/events use the same code to view and edit.

However, after checking your issue, I recognized that the copy icon is shown for source citations within NAME, but the paste icon cannot be used. This is really inconsistent.

copy paste source citations in INDI-NAME

The reason is that a source citation within NAME is shown with the same code like within other facts/events. Therefore, the copy icon is shown (although not intended). However, the name data itself is shown with another piece of webtrees code, which is different from facts/events. Therefore, the paste icon is not shown here.

I have to think about this issue and will check how much effort it is to add the paste icon to NAME also.

@HerzScheisse
Copy link
Contributor Author

yeah i noticed that...

no worries... its not that urgent...
at the moment i'm still using raw gedcom to add source citations, as i have been using since my start with webtrees...

only the missing paste ability for the name fields stop me from using this function in general...

@Jefferson49
Copy link
Owner

I was able to include the paste mechanism into NAME. However, there is an issue with vesta. If using vesta in parallel, the paste icon is not shown. The reason is that vesta modifies the NAME view also and the two modifications will not be combined. From your screenshot, I assume that you are using vesta.

Without vesta it looks like this:

paste icon NAME

If you want to test the paste mechanism in NAME, you can install the attached module, which is version v1.38 plus the paste mechanism in NAME:

repository_hierarchy_v1.3.8_f16987b2.zip

@HerzScheisse
Copy link
Contributor Author

thanks for the addition... :)

@ric2016
Copy link

ric2016 commented Feb 4, 2024

Ideally there would be another sub-view in webtrees just for the edit controls of a name. Based on past experience I do not think such a feature request would be accepted though.

I'll expand the Vesta configuration options instead.

@Jefferson49
Copy link
Owner

I'll expand the Vesta configuration options instead.

Hallo Richard,

thank you very much for supporting this issue! I agree that we should not create dependencies between the custom modules. I think that the option in vesta is a good approach.

@HerzScheisse
Copy link
Contributor Author

wow that was a fast fix... thank both of you for your changes :)

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

No branches or pull requests

3 participants