You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The presentation and look of the calendar is pretty slick, but there should be an option to position the calendar based on the button. Like above or below from the button, or left or right. When I used my version of the calendar, I wanted it below and right-aligned with the button, so I changed the percentage value of the contents-wrapper, like so:
It works, but it should rather be a prop. We use the calendar multiple times and each of them isn't meant to be in this position. If this would become an enchancement, the button should also close the calendar.
The text was updated successfully, but these errors were encountered:
That's an interesting idea. The way the popover component works right now is intentional but this might be a nice enhancement (I've labeled it as such). I'm not sure it's as simple as allowing that position to be set as a prop, though, as the component does do some viewport boundary detection to make sure it does not position itself partially/completely out of the viewport. It's also worth noting that the popover positions differently on mobile (to be closer to the user's thumbs). I'm not sure how this would interact with any sort of positioning prop/option.
Just wanted to say that we also need a feature like this, since we are now embedding iframes in client pages, and the viewport is the whole iframe, not the browser.
The presentation and look of the calendar is pretty slick, but there should be an option to position the calendar based on the button. Like above or below from the button, or left or right. When I used my version of the calendar, I wanted it below and right-aligned with the button, so I changed the percentage value of the contents-wrapper, like so:
It works, but it should rather be a prop. We use the calendar multiple times and each of them isn't meant to be in this position. If this would become an enchancement, the button should also close the calendar.
The text was updated successfully, but these errors were encountered: