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 "Manage Tags..." context menu option #2716

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AbeJellinek
Copy link
Member

Fixes #2014.

Do we want a keyboard shortcut too? Or is that Zutilo territory?

chrome/locale/en-US/zotero/zotero.properties Outdated Show resolved Hide resolved
chrome/content/zotero/zoteroPane.js Outdated Show resolved Hide resolved
chrome/content/zotero/zoteroPane.js Outdated Show resolved Hide resolved
This reworks tagsBox.js a bit, but behavior should be exactly the same
when only viewing one item (which is always the case in the item box).
The external API remains the same if you're just setting/retrieving a
single item.
@AbeJellinek AbeJellinek changed the title Add "Add Tag..." context menu option Add "Manage Tags..." context menu option Jul 22, 2022
@AbeJellinek
Copy link
Member Author

OK, reworked into something quite a bit fancier using the tags box CE.

(Tags box supporting multiple items might be helpful if we ever decide to implement batch item editing.)

@AbeJellinek
Copy link
Member Author

Looks like this:

image

  1. Is this along the lines of what we'd want?
  2. Should it update if you select a new item/set of items, Quick Look-style?
  3. Should it have a close button or is pressing Escape obvious enough?

@zotero zotero deleted a comment from Ray-Eldath Oct 31, 2022
@dstillman dstillman deleted the branch zotero:main May 21, 2023 22:54
@dstillman dstillman closed this May 21, 2023
@dstillman dstillman reopened this May 22, 2023
@dstillman dstillman changed the base branch from fx102 to main May 22, 2023 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants