Skip to content

Nanopop update issue when using with Pickr and large screen sizes #231

@alexdiliberto

Description

@alexdiliberto

First off, just wanted to say I'm really enjoying your Pickr library.

I do have an issue I'm running into on my app that seems to be Nanopop/positioning related. I wasn't sure which library to raise the issue (Pickr or Nanopop) but I'll try here. I have a minimal reproduction on this Codepen:

https://codepen.io/alexdiliberto/pen/vYLrxyq

You can see the issue if you start resizing your screen larger and larger then it seems to begin throwing off the positioning until it finally fails.

1680px screen width (expected behavior)

Screen Shot 2020-07-10 at 11 51 24 AM

2300px screen width

Screen Shot 2020-07-10 at 11 51 39 AM

2450px screen width

Screen Shot 2020-07-10 at 11 54 45 AM

2600px screen width

Screen Shot 2020-07-10 at 11 51 45 AM

eventually on large screens it seems to fail this conditional and just places the pickr app in the center of the screen

Any assistance would be greatly appreciated, none of the public API's for Pickr helped in this case...I manually set this._recalc (https://github.com/Simonwep/pickr/blob/1.7.1/src/js/pickr.js#L513) to false and that seemed to help by just now allowing the final fail case where it places the pickr app in the center of the screen.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions