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 share button with citation hint to TIDO #470

Open
mokkalicious opened this issue Sep 12, 2024 · 8 comments
Open

Add share button with citation hint to TIDO #470

mokkalicious opened this issue Sep 12, 2024 · 8 comments

Comments

@mokkalicious
Copy link
Collaborator

mokkalicious commented Sep 12, 2024

Description:

This issue was noticed in Ahiqar project. --> https://projects.academiccloud.de/wp/42467
In TIDO Simon suggested a share button with the citation hint.

This is about the citation hint within the TIDO Metadata tab as seen in this example:
Image

Acceptance Criteria

Share button is located in position which makes sense UX wise

MVP: Share button copies citation hint to clipboard

_Nice2Have: button opens dialogue to choose between options like, e.g.

copy to clipboard

open in your email software

open in Zotero(?)_

@paulpestov
Copy link
Collaborator

How should we identify that it's a citation? If I remember correctly that part is a dynamic key/value pair, so projects might use a different key for their citations like "Zitation" or "Zitat" or "How to Cite". I see two options:

@orlinmalkja orlinmalkja self-assigned this Oct 9, 2024
@mathias-goebel
Copy link
Member

to generalize this, a "share" button might become a "copy" button and can be available to all fields. to keep the UI clean, a button might become available on hover, since a user who wants to copy the citation statement will always move a cursor to the respective field

@mweidling
Copy link
Contributor

to generalize this, a "share" button might become a "copy" button and can be available to all fields.

Good idea!

a button might become available on hover

There is at least one project where we plan for tablet usage. Hovering will not cover this.

@mathias-goebel
Copy link
Member

mathias-goebel commented Nov 21, 2024

Hovering will not cover this.

I think it will. Mobile Browsers will keep :hover when taped on a shape.

@paulpestov
Copy link
Collaborator

paulpestov commented Nov 21, 2024

Not sure about displaying the copy button everywhere. What about values that contain dynamic HTML or Markdown (that decision that we still have to make)? We currently have a PR for the citation key solution in the config. #530

@orlinmalkja
Copy link
Contributor

Hi guys
I am not sure either of displaying the copy button in every metadata item.

in the current PR #530 a solution to this issue is done according to 'Make it configurable as metadata panel option, e.g. citationKey = "Citation"'.
The copy button is shown only in the 'row' when there is a config value of citation the same as TextApi metadata key 'Citation'.

@mweidling
Copy link
Contributor

@mathias-goebel, @stexandev and I discussed about this. Our solution will be as follows:

  • there will be a separate field in the Support Object for a CFF file. TIDO can then take this file and render it in different formats (e.g. Chicago style, BibTeX, …)
  • since the Support Object is on Manifest level, the Item will be considered in the URL present in the citation. This URL has to be fetched and inserted by the client.
  • We agree on making the citation key configurable in the metadata panel options

@paulpestov
Copy link
Collaborator

The discussion that goes beyond the citation key should be in a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants