[GH-ISSUE #1428] Is the port number really more readable with a , ? #1421

Open
opened 2026-03-03 19:51:22 +03:00 by kerem · 8 comments
Owner

Originally created by @ekscrypto on GitHub (Nov 7, 2022).
Original GitHub issue: https://github.com/ProxymanApp/Proxyman/issues/1428

Originally assigned to: @NghiaTranUIT on GitHub.

Description

When hovering the "Proxy overriden" pill on the bottom right it indicates the IP address and the port number. However that port number now has a thousand separators which in this case seems to me more confusing then helping, since you shouldn't be entering that , on the iOS/Android proxy settings.

Originally created by @ekscrypto on GitHub (Nov 7, 2022). Original GitHub issue: https://github.com/ProxymanApp/Proxyman/issues/1428 Originally assigned to: @NghiaTranUIT on GitHub. ## Description When hovering the "Proxy overriden" pill on the bottom right it indicates the IP address and the port number. However that port number now has a thousand separators which in this case seems to me more confusing then helping, since you shouldn't be entering that , on the iOS/Android proxy settings.
Author
Owner

@NghiaTranUIT commented on GitHub (Nov 7, 2022):

Thousand separator? Can you share with me the screenshot 🤔

<!-- gh-comment-id:1305815643 --> @NghiaTranUIT commented on GitHub (Nov 7, 2022): Thousand separator? Can you share with me the screenshot 🤔
Author
Owner

@delboy1978uk commented on GitHub (Nov 7, 2022):

@NghiaTranUIT
image

<!-- gh-comment-id:1305910244 --> @delboy1978uk commented on GitHub (Nov 7, 2022): @NghiaTranUIT <img width="687" alt="image" src="https://user-images.githubusercontent.com/2684575/200369850-0b6211a5-e969-4b8d-a4c4-3633c23cc1a1.png">
Author
Owner

@NghiaTranUIT commented on GitHub (Nov 7, 2022):

Ah, thanks for the screenshot.it's a UI localized bug. Let me fix it 👋

<!-- gh-comment-id:1306138367 --> @NghiaTranUIT commented on GitHub (Nov 7, 2022): Ah, thanks for the screenshot.it's a UI localized bug. Let me fix it 👋
Author
Owner

@NghiaTranUIT commented on GitHub (Nov 8, 2022):

It's fixed @delboy1978uk @ekscrypto https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.13.0_Fix_Port_string_format.dmg

Screen Shot 2022-11-08 at 20 26 17

This bug is accidentally introduced when Proxyman supports Localization. String.localizedStringWithFormat() did use a thousand separators on the number. Fix it by converting to a string, not a number 👍

<!-- gh-comment-id:1307247467 --> @NghiaTranUIT commented on GitHub (Nov 8, 2022): It's fixed @delboy1978uk @ekscrypto https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.13.0_Fix_Port_string_format.dmg <img width="722" alt="Screen Shot 2022-11-08 at 20 26 17" src="https://user-images.githubusercontent.com/5878421/200580930-d18fde35-7ed4-4f42-b495-0a9cc831a1ee.png"> This bug is accidentally introduced when Proxyman supports Localization. `String.localizedStringWithFormat()` did use a thousand separators on the number. Fix it by converting to a string, not a number 👍
Author
Owner

@abardallis commented on GitHub (Nov 17, 2022):

FWIW, same thing in the macOS menu bar:

Proxyman menu bar menu

<!-- gh-comment-id:1318870411 --> @abardallis commented on GitHub (Nov 17, 2022): FWIW, same thing in the macOS menu bar: ![Proxyman menu bar menu](https://user-images.githubusercontent.com/9538506/202498500-eba16111-7e95-4a9f-a89b-f101ceeda824.png)
Author
Owner

@NghiaTranUIT commented on GitHub (Nov 18, 2022):

Sorry for the inconvenience. I incorrectly used String.localizedStringWithFormat() again.

Here is the Fix: https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.13.0_Fix_ip_port_localization_again.dmg

Screenshots

Screen Shot 2022-11-18 at 09 01 07 Screen Shot 2022-11-18 at 09 00 59
<!-- gh-comment-id:1319451439 --> @NghiaTranUIT commented on GitHub (Nov 18, 2022): Sorry for the inconvenience. I incorrectly used `String.localizedStringWithFormat()` again. Here is the Fix: https://proxyman.s3.us-east-2.amazonaws.com/beta/Proxyman_3.13.0_Fix_ip_port_localization_again.dmg ### Screenshots <img width="327" alt="Screen Shot 2022-11-18 at 09 01 07" src="https://user-images.githubusercontent.com/5878421/202599590-99394d67-6c47-4149-9cd5-a2ab4bbebe28.png"> <img width="658" alt="Screen Shot 2022-11-18 at 09 00 59" src="https://user-images.githubusercontent.com/5878421/202599603-d9357bf9-a315-4ed8-a43b-587ada23b5c7.png">
Author
Owner

@delboy1978uk commented on GitHub (Nov 18, 2022):

You are awesome! Thank you! I reaaalllly like Proxyman, you have made some incredible software! Have a great day!

<!-- gh-comment-id:1319792881 --> @delboy1978uk commented on GitHub (Nov 18, 2022): You are awesome! Thank you! I reaaalllly like Proxyman, you have made some incredible software! Have a great day!
Author
Owner

@abardallis commented on GitHub (Nov 18, 2022):

No inconvenience at all @NghiaTranUIT – Thank you!
Installed the build you just linked and it looks to be fixed.

Also +1 to you being awesome, making incredible software, and me really liking Proxyman 👍

<!-- gh-comment-id:1320082749 --> @abardallis commented on GitHub (Nov 18, 2022): No inconvenience at all @NghiaTranUIT – Thank you! Installed the build you just linked and it looks to be fixed. Also +1 to you being awesome, making incredible software, and me really liking Proxyman 👍
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Proxyman#1421
No description provided.