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

Port is limited to 4 nums #95

Closed
DoumanAsh opened this issue Jan 19, 2016 · 3 comments · Fixed by #100
Closed

Port is limited to 4 nums #95

DoumanAsh opened this issue Jan 19, 2016 · 3 comments · Fixed by #100
Labels
🐛 bug Bugs. We have them.

Comments

@DoumanAsh
Copy link
Contributor

While port can be up to 65k, config allows you to enter only 4 num.
Funny thing is that you can enter some non-numeric character :)

p.s. manual edit of config works of course :)

@zp
Copy link
Contributor

zp commented Jan 19, 2016

Which version are you using? v0.4 appears to handle 5 digit ports fine via preferences window. (for me at least)

It does let you input values lower/higher than 1024/65535 though, not sure how the back-end handles that. (picotorrent crashes for me if i attempt to use letters)

@vktr vktr added the 🐛 bug Bugs. We have them. label Jan 19, 2016
@vktr
Copy link
Member

vktr commented Jan 19, 2016

Thanks for reporting! Are you running a retina display? I believe I've seen this with retina displays, that they size the input a bit too small (even though I do DPI scaling) and then only accept four digits.

Input validation is missing, though 😅

@DoumanAsh
Copy link
Contributor Author

Example
http://i.imgur.com/EmBhLxg.png

Well not sure what is type of display i'm having :D
I cannot put 5, but i can inputs
P.s. i'm using latest release build
p.s.s. yes, i suppose the problem may be in width of input. Which is actually quite uncommon for me to be a limit :D

@vktr vktr mentioned this issue Jan 20, 2016
3 tasks
@vktr vktr closed this as completed in #100 Jan 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Bugs. We have them.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants