fix tree detail height#62
Conversation
| <IonCard | ||
| style={{ | ||
| position: "absolute", | ||
| margin: "0px", |
There was a problem hiding this comment.
The card has a margin, that was screwing up on iPads due to different values in relation to current viewport size
| margin: "0px", | ||
| top: 60, | ||
| left: 12, | ||
| left: 20, |
There was a problem hiding this comment.
This is the absolute displacement of the Buttons above the details
| width: "400px", | ||
| height: "90%", | ||
| maxWidth: "80%", | ||
| height: "calc(100% - 60px - 20px)", |
There was a problem hiding this comment.
the height is now calculated for the current height minus the fixed offsets
| // width: "33vw", | ||
| width: "400px", | ||
| height: "90%", | ||
| maxWidth: "80%", |
There was a problem hiding this comment.
Makes the card slightly smaller on iPad to not overlay the position button on the map
|
For iPad, the card is still too high (Brave) |
Can you also try chrome or safari? On both, a iPad screen size works for me, or did you try an actual iPad? |
JesJehle
left a comment
JesJehle
left a comment
There was a problem hiding this comment.
It's working now. No idea why ...
2 participants
fixes #58
I tested for Desktop, Mobile and iPad Air (in Chrome) to fix the width of the Tree Detail card.
I am not sure, what exactly the problem is on mobile devices. I will add a commented review to explain my changes.