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

#2417 Update the tooltip for 'Anyone link' #2488

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

SebinSong
Copy link
Collaborator

closes #2417

Along with the copy update, I ensured the tooltip is anchored to the trigger element too, just like we did in #2482

[When not enough space - position it to the right]

@SebinSong SebinSong self-assigned this Dec 30, 2024
Copy link

cypress bot commented Dec 30, 2024

group-income    Run #3637

Run Properties:  status check passed Passed #3637  •  git commit 56594d2253 ℹ️: Merge db7da351ca7241184a15d8986e60de51596a2db8 into 160bb6231efbcee1834b96106dc0...
Project group-income
Branch Review sebin/task/#2417-anyone-link-tooltip-update
Run status status check passed Passed #3637
Run duration 10m 38s
Commit git commit 56594d2253 ℹ️: Merge db7da351ca7241184a15d8986e60de51596a2db8 into 160bb6231efbcee1834b96106dc0...
Committer Sebin Song
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 10
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 112
View all changes introduced in this branch ↗︎

@@ -22,7 +22,7 @@ page-section.c-section(:title='L("Invite links")')
i18n.c-invite-link(tag='th') invite link
i18n.c-state(tag='th') state
th.c-action(
:aria-label='L("action")'
:aria-label='L("action ")'
Copy link
Member

Choose a reason for hiding this comment

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

This looks like a mistake

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

Successfully merging this pull request may close these issues.

Change tooltip for "Anyone" link
2 participants