Skip to content

Add openTab in context menu#73

Open
ramonmi wants to merge 2 commits into
zotero:masterfrom
ramonmi:add-open-note-in-tab-context-menu
Open

Add openTab in context menu#73
ramonmi wants to merge 2 commits into
zotero:masterfrom
ramonmi:add-open-note-in-tab-context-menu

Conversation

@ramonmi
Copy link
Copy Markdown
Contributor

@ramonmi ramonmi commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

@windingwind windingwind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good to me

Comment thread src/ui/toolbar.js
{l10n.getString('general-show-in-library')}
</button>
)}
{viewMode !== 'tab' && viewMode !== 'ios' && (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this should apply to android as well

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have checked the latest release version on my Android tablet, and there is no OpenWindow button in the toolbar. So I suppose this is inessential.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, there is a no-op option for Android with the missing Fluent string.

@ramonmi ramonmi force-pushed the add-open-note-in-tab-context-menu branch from fca6ced to a42a4c7 Compare March 4, 2026 01:18
@ramonmi ramonmi force-pushed the add-open-note-in-tab-context-menu branch from a42a4c7 to 102d76f Compare April 9, 2026 12:15
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.

3 participants