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

[PLAYER-31] fix wrong calculation border with css scale rule #91

Merged

Conversation

jparez
Copy link
Contributor

@jparez jparez commented Aug 6, 2024

Description

When a css rules (i.e. scale) is set on the player's HTML element the calculation of borders is wrong. This is caused by the offset's browser api which doesn't take css rules in this calculation

  • change way of borders are calculated
  • fix bug on keymapping debug grids for portrait

@jparez jparez force-pushed the dev/PLAYER-31-fix-wrong-calculation-border-with-css-scale-rule branch from 226ded4 to a293e23 Compare August 7, 2024 11:24
@jparez jparez force-pushed the dev/PLAYER-31-fix-wrong-calculation-border-with-css-scale-rule branch from a293e23 to 9ea4dc6 Compare August 7, 2024 11:31
@jparez jparez requested a review from pgivel August 7, 2024 11:32
Copy link
Contributor

@pgivel pgivel left a comment

Choose a reason for hiding this comment

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

Nice !

another solution could have been to simply always remove the top border from the height and the left border from the width I guess (times two of course)

@jparez jparez merged commit 3c0e4de into main Aug 7, 2024
1 check passed
@jparez jparez deleted the dev/PLAYER-31-fix-wrong-calculation-border-with-css-scale-rule branch August 7, 2024 13:10
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.

2 participants