Skip to content

fix: [M3-7149] - Fix textfield tooltip icon focus area#10938

Merged
hana-akamai merged 2 commits intolinode:developfrom
hkhalil-akamai:M3-7149-tooltip-focus
Sep 17, 2024
Merged

fix: [M3-7149] - Fix textfield tooltip icon focus area#10938
hana-akamai merged 2 commits intolinode:developfrom
hkhalil-akamai:M3-7149-tooltip-focus

Conversation

@hkhalil-akamai
Copy link
Contributor

Description 📝

Fixes the focus area for tooltips to be a circle rather than an oval as before.

Preview 📷

Before After
Screenshot 2024-09-13 at 5 21 43 PM Screenshot 2024-09-13 at 5 21 32 PM

How to test 🧪

Verify textfield tooltips' focus areas render as a circle and otherwise work as before.

@hkhalil-akamai hkhalil-akamai self-assigned this Sep 13, 2024
Comment on lines +482 to +485
height: '34px',
margin: '0px 0px 0px 4px',
padding: '6px',
padding: '17px',
width: '34px',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Specifying these as numbers instead of '__px' changes the appearance of the tooltip icon

@hkhalil-akamai hkhalil-akamai marked this pull request as ready for review September 13, 2024 21:27
@hkhalil-akamai hkhalil-akamai requested a review from a team as a code owner September 13, 2024 21:27
@hkhalil-akamai hkhalil-akamai requested review from carrillo-erik and hana-akamai and removed request for a team September 13, 2024 21:27
@hana-akamai hana-akamai added the Add'tl Approval Needed Waiting on another approval! label Sep 16, 2024
@coliu-akamai coliu-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Sep 17, 2024
@hana-akamai hana-akamai merged commit 3758ec1 into linode:develop Sep 17, 2024
@hkhalil-akamai hkhalil-akamai deleted the M3-7149-tooltip-focus branch September 19, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants