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
Fixed tooltips for element links #12840
Conversation
I cannot get the annoying tail to sit correctly. I am not even sure where it is actually supposed to go? Middle of the tooltip or middle of the element?
This comment was marked as resolved.
This comment was marked as resolved.
Plugin builds for 913d571 are ready 🛎️!
|
Size Change: +301 B (0%) Total Size: 2.7 MB
ℹ️ View Unchanged
|
expect(withProtocol('https://foo.com/test')).toBe('https://foo.com/test'); | ||
expect(withProtocol('http://foo.com/test')).toBe('http://foo.com/test'); | ||
expect(withProtocol('foo.com/test')).toBe('https://foo.com/test'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only reason for this change is to make each fit on a single line because it's prettier 😂
Confirmed that element links attached to the following different types of elements yields favorable results with the
I also tried this on a mobile device, and desktop browsers. |
@swissspidy @barklund - the observation I wanted to share with you is that if I attach a link to a text element, then the tail of the tool tip is sometimes difficult to see since the color of the tool tip matches the (default) background color -- white |
Context
This is an attempt at fixing the tooltips.
To-do
User-facing changes
Testing Instructions
This PR can be tested by following these steps:
Checklist
Type: XYZ
label to the PRFixes #12834