Opened 20 years ago
Closed 20 years ago
#663 closed patch (wontfix)
Tidy up DVB card setup
Reported by: | Owned by: | danielk | |
---|---|---|---|
Priority: | minor | Milestone: | 0.19 |
Component: | mythtv | Version: | head |
Severity: | medium | Keywords: | DVB setup |
Cc: | Ticket locked: | no |
Description
I couldn't read the DVB card setup page. So, the attached patch :
disables Disqec button non QSPK DVB cards
move (msecs) into the help text for the timeout vales so the controls fit on screen.
It should be noted that the help text could do with being a bit shorter :)
Attachments (2)
Change History (9)
by , 20 years ago
Attachment: | videosouce.patch added |
---|
comment:1 by , 20 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 by , 20 years ago
Daniel, I'm a bit concerned you've missed the main point of the patch :) The labels for the two spin boxes were/are too long (didn't fit at all well). I'll Test it tonight when I get home and let you know.
comment:3 by , 20 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I was wrong it looks fine, apart from the minor addendum of the DiSEqC button being active for non QPSK card types. (patch attached)
by , 20 years ago
Attachment: | videosource.patch added |
---|
comment:4 by , 20 years ago
Milestone: | → 0.19 |
---|---|
Version: | → head |
I put the two spinboxes on seperate lines so the label could still show the units.
I'm not sure if we wan't to disable DiSEqC just based on the modulation type.
Isn't there some way we can query whether the device is DiSEqC capable?
Partially my concern is for the pcHDTV HD-2000. It has two inputs, but the DVB API only supports multiple inputs via DiSEqC. So unless the DVB API is changed, a card like this can only be supported by emulating a DiSEqC device in the driver. (Yes, the DVB API is a mess on every level and could use a complete rewrite, but I can only fight so many windmills.)
comment:5 by , 20 years ago
I hadn't even considered that. I assumed it was only sat stuff which needed DiSEqC. I've certainly never heard of dvb-T/C ATSC needing it but that could just be my ignorance :)
comment:6 by , 20 years ago
I'm going to close this for now; I'll look at the drivers and make it right, whatever that might be, after 0.19.
comment:7 by , 20 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
(In [7898]) Fixes #663.
I didn't use the patch but tried to address the concerns of overly verbose help text, and cramped layout. I also brought some consistency to the "DiSEqC" wacky mixed case string.