🐞 Weird text size constraint in the issue number of the update window

[hiraginoyuki]

Bug Description

In The Loop Times window, there is a slight issue where the displayed issue number is wrapped at the fourth digit (which, on the bright side, is an amazing sign that this project has received a TON of contributions!)

Let me know if there's something wrong with the format of this issue, as this is the first time contributing to this repo (despite using the actual software for over an year 😂)

Affected Scope

User interface

Steps to Reproduce

1. Wait for Loop to announce an update (The Loop Times)

Reproducibility

Always

Expected vs Actual Behavior

Expected: The text is not wrapped.

Actual: The text is wrapped.

Screen Recordings / Screenshots

Severity

Trivial (cosmetic)

macOS Version

26.1 Beta (25B5072a)

Loop Version

🧪 1.4.1 (1683)

Did You Try the Development Build?

Yes

Additional Context

• The main debate imo is which approach we should take, i.e we can make it so that:
  • the text is wrapped around at the next digit, which makes it work until the issue number exceeds 10k, or
  • we can make it so that the text is NEVER wrapped (which I currently don't know how, as I haven't investigated the actual code yet)

Final Checks

• My issue is written in English.
• My issue title is descriptive.
• This is a single bug (multiple bugs should be reported individually).
• I have looked to see if this is a duplicate of another bug report.
• I can help with further investigation.
• I can help with developing a fix for this issue.

[hiraginoyuki]

This is the section that has the width constraint.

https://github.com/MrKai77/Loop/blob/a80292f7023e194bd5ebc6b85815f1c0e5e36281/Loop/Updater/UpdateView.swift#L318-L323

[hiraginoyuki]

Staring further at the code, I just realized that the entire Updater window's width is determined by the issue section...? That's kind of hilarious because my intuition regarding UIs is that everything is kinda flex-like...

On the contrary, that fact makes it easier to fix this issue because we can just blindly increase the number and call it a day? lol

[mrkai77]

Thanks for reporting this! I probably wouldn’t have noticed it for a long time otherwise, since I usually run my own debug builds 😅

The main debate imo is which approach we should take, i.e we can make it so that:

• the text is wrapped around at the next digit, which makes it work until the issue number exceeds 10k, or
• we can make it so that the text is NEVER wrapped (which I currently don't know how, as I haven't investigated the actual code yet)

I think the second option makes most sense. You can prevent text from wrapping using the .fixedSize() modifier, and also removing the existing .frame(width:) modifier.

Staring further at the code, I just realized that the entire Updater window's width is determined by the issue section...? That's kind of hilarious because my intuition regarding UIs is that everything is kinda flex-like...

https://github.com/MrKai77/Loop/blob/a80292f7023e194bd5ebc6b85815f1c0e5e36281/Loop/Updater/UpdateView.swift#L71

This should be the code to set the update window's size :)

Would you like to open a PR for this?

[mrkai77]

Fix has been implemented in #1020

[hiraginoyuki]

Oh... guess I was too late 😅 but happy that it's fixed regardless!