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

Citation dialog: add a dropmarker to bubbles #5130

Merged
merged 2 commits into from
Mar 21, 2025

Conversation

abaevbog
Copy link
Contributor

  • add dropmarker to bubbles after the text to indicate that bubbles are clickable
  • remove X remove button that was there before
  • no more mask over the bubble on hover, since dropmarker is just always visible
  • 4px distance between the dropmarker and bubble text

Fixes: #2712

Screenshot 2025-03-18 at 11 26 52 PM

- add dropmarker to bubbles after the text to indicate
that bubbles are clickable
- remove X remove button that was there before
- no more mask over the bubble on hover, since dropmarker
is just always visible
- 4px distance between the dropmarker and bubble text

Fixes: zotero#2712
@abaevbog
Copy link
Contributor Author

@yexingsha how is this tweak looking to you? Here, the X button is gone and instead we have a chevron-6 dropmarker 4px after the bubble's text. It is always visible, so the the mask effect and etc. on hover is removed too.

@yexingsha
Copy link
Contributor

Looks good to me!

@abaevbog abaevbog requested a review from dstillman March 19, 2025 21:39
@dstillman
Copy link
Member

The dropmarker goes away if you customize the citation (and even if you remove the customization):

Screenshot 2025-03-20 at 2 44 21 AM

Screenshot 2025-03-20 at 2 44 30 AM

@abaevbog
Copy link
Contributor Author

Sorry about that! Should be fixed now:

Screenshot 2025-03-20 at 8 56 01 AM

@dstillman dstillman merged commit 41bb278 into zotero:main Mar 21, 2025
1 check passed
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.

Citation dialog: Add a dropmarker to citation bubbles
3 participants