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

Using Wrap to allow longer pin codes to wrap into another row #165

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NunoOliveiraP
Copy link

We love Pinput's UI and capabilities for entering typical 4-6 length pin codes. However, when faced with a very large key codes which are typical used for activating product keys or gift codes, it forces every input to stay inside a single row... With a 16-length gift code like we use, plus a few - to separate the code for easier readability, it makes it impossible to achieve a pleasant UI.

After checking the package code for a bit I experimented with switching the Row widget which holds the inputs to a Wrap. It holds the same row-like structure up to the point where it needs to shrink. It is also very customizable, which is good for fitting into various UI layouts.

It worked perfectly for us (as you can see in the screenshots below) and we're wondering if this would be a good addition to the package itself. Perhaps adding a 2nd widget or constructor which uses Wrap instead of Row is preferable, but I think that is up to the package owner to decide. Here are the small changes we did in order to make it work.

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.

1 participant