Skip to content

Fix wrong position of tooltip when overflow screen width#114

Merged
jasongaare merged 2 commits intojasongaare:masterfrom
serzmerz:master
Feb 16, 2021
Merged

Fix wrong position of tooltip when overflow screen width#114
jasongaare merged 2 commits intojasongaare:masterfrom
serzmerz:master

Conversation

@serzmerz
Copy link
Copy Markdown
Contributor

@serzmerz serzmerz commented Feb 11, 2021

This PR fix issues:
Fixes #78
Fixes #82

The problem that when the tooltip overflow screen width, it takes the center of the screen, but should right side.

Before:
Simulator Screen Shot - iPhone 11 - 2021-02-11 at 16 29 07

After:
Simulator Screen Shot - iPhone 11 - 2021-02-11 at 16 28 27

@serzmerz
Copy link
Copy Markdown
Contributor Author

serzmerz commented Feb 12, 2021

@jasongaare Please make a review of these changes

@jasongaare
Copy link
Copy Markdown
Owner

awesome! this looks great. let me test locally then we can merge!

@jasongaare jasongaare merged commit e91f277 into jasongaare:master Feb 16, 2021
@jasongaare
Copy link
Copy Markdown
Owner

Awesome! Published in version 1.2.0

@jasongaare jasongaare mentioned this pull request Mar 16, 2021
@joluet
Copy link
Copy Markdown

joluet commented Apr 28, 2021

This actually breaks the centering of tooltips when the content overflows screen width. Such tooltips are always right aligned now.

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.

Tooltip defaults to center of screen if surpasses displayInset on the right side Need to offset the tooltip depending on size of tooltip

3 participants