Opened 15 years ago
Closed 14 years ago
#6106 closed defect (invalid)
LCDProc: Scrolling menus on small LCD displays lack first 3 characters
Reported by: | Owned by: | paulh | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | MythTV - Mythlcdserver | Version: | head |
Severity: | low | Keywords: | LCDproc menu scroll |
Cc: | Ticket locked: | no |
Description
Context: When menus are shown on a LCDProc display, they can scroll to the left when the text is too long for the width of the display. And the menu texts can be preceded by " ", "> " or "X " based on selected and checked status. In smaller LCD displays (<= 12 chars) the spaces are to be trimmed here. Observed: Menu ">Manage Recordin" changes to "> anage Recordin" when it starts scrolling. The extra spaces are not trimmed here.
Attachments (2)
Change History (9)
Changed 15 years ago by
Attachment: | scrolldiff.out added |
---|
comment:1 Changed 15 years ago by
The issue is a bit different than I thought at first sight. The root cause was that:
- global (??!!) var lcdStartCol was only initialized in the constructor of LCDProcClient, before the display width is set to the correct value;
- De different locations where the menu text is changed used different conditions to select the prefix of the menu text.
Both are fixed in the attached scrolldiff.out
comment:2 Changed 15 years ago by
Status: | new → infoneeded_new |
---|
Please can you separate your patches? This one appears to be including the priority fix as well.
comment:3 Changed 15 years ago by
Status: | infoneeded_new → new |
---|
comment:4 Changed 15 years ago by
Component: | mythtv → MythTV - Mythlcdserver |
---|---|
Milestone: | unknown → 0.22 |
Owner: | changed from Isaac Richards to paulh |
Status: | new → assigned |
Version: | 0.21-fixes → head |
comment:6 Changed 14 years ago by
Milestone: | 0.23 → unknown |
---|
Can you please update this patch for current trunk.
comment:7 Changed 14 years ago by
Resolution: | → invalid |
---|---|
Status: | assigned → closed |
No reply in 6 weeks. Please reopen when you have an updated patch.
Patch to consequently add (or don't add) spaces on menus