Opened 17 years ago
Closed 17 years ago
#2201 closed bug (fixed)
[PATCH] Disc Time Label (fDiscTime) not long enough
| Reported by: | sbrokenshire | Owned by: | darkwyrm |
|---|---|---|---|
| Priority: | normal | Milestone: | R1 |
| Component: | Applications/CDPlayer | Version: | R1/pre-alpha1 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
Using VMWare, Haiku SVN hrev25302
This is reproduced by starting CDPlayer, inserting an Audio CD (over 10 minutes long) and playing it.
Disc Time Label (fDiscTime) shows the current and total disc time but total disc time is partially shown because fDiscTime is not long enough. Expected both current and total disc time to be shown in full.
Patch attached.
Attachments (1)
Change History (2)
by , 17 years ago
| Attachment: | CDPlayer.fDiscTimenotlongenough.diff added |
|---|
comment:1 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Fixed in hrev25412. I did not use your patch, because it was just another way of hardcoding the size, and it also didn't fix the potential track time truncation either. The problem was that the size calculation was based on the wrong string with regards to proportional fonts.
Note:
See TracTickets
for help on using tickets.
Patch for fDiscTime in CDPlayer