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

Set default tooltip placement to be top #4010

Merged
merged 2 commits into from Jul 4, 2023

Conversation

ivarnakken
Copy link
Member

Description

When there is space, I prefer to have it on top. It's also the more "common" placement.

Result

Screen.Recording.2023-06-25.at.12.55.06.mov

Testing

  • I have thoroughly tested my changes.

It still works. See video above.

@ivarnakken ivarnakken added enhancement Pull requests that make enhancements, instead of just purely new features review-needed Pull requests that need review labels Jun 25, 2023
@ivarnakken ivarnakken requested a review from a team June 25, 2023 10:57
When there is space, I prefer to have it on top. It's also the more
"common" placement.
@ivarnakken ivarnakken force-pushed the set-default-tooltip-placement branch from 4df8abc to 5b4768c Compare July 2, 2023 16:48
@ivarnakken ivarnakken added the approved Pull requests that have been approved label Jul 2, 2023
@ivarnakken ivarnakken added ready-to-merge Pull requests that have been approved and are ready to be merged and removed review-needed Pull requests that need review labels Jul 4, 2023
@ivarnakken ivarnakken enabled auto-merge July 4, 2023 15:17
@ivarnakken ivarnakken merged commit 0298928 into master Jul 4, 2023
4 checks passed
@ivarnakken ivarnakken deleted the set-default-tooltip-placement branch July 4, 2023 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved enhancement Pull requests that make enhancements, instead of just purely new features ready-to-merge Pull requests that have been approved and are ready to be merged
Projects
None yet
3 participants