Opened 17 years ago
Closed 16 years ago
Last modified 16 years ago
#3797 closed enhancement (fixed)
Gray letterboxing option
Reported by: | anonymous | Owned by: | danielk |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythtv | Version: | head |
Severity: | low | Keywords: | gray pillarbars |
Cc: | Ticket locked: | no |
Description
I hate watching programs in the wrong aspect ratio or parts cut off in a zoom mode, but I didn't want to damage my RP set with the black pillarboxes currently provided with Aspect Override turned off. Here is a patch that adds a Gray Pillarboxes option to the Aspect Override menu option. It should perform identical to the Off setting, just with the pillarboxes gray instead of black.
Attachments (4)
Change History (9)
Changed 17 years ago by
Attachment: | pillarbars.diff added |
---|
comment:1 Changed 17 years ago by
Milestone: | 0.21 → unknown |
---|---|
Owner: | changed from Isaac Richards to danielk |
Changed 17 years ago by
Attachment: | pillarbars.2.diff added |
---|
comment:2 Changed 17 years ago by
I didn't like it hooked to the aspect override either after I thought about it some more. Now it is a seperate setting under TV Settings, Playback. The menu part you still won't like, but the second patch does require a field in settings named GrayPillars?, 0 for off, 1 for on, so it can be toggled without having to recompile.
I tried to replace the first patch, but was told no. If you could please remove that one for me that would be appreciated.
Changed 17 years ago by
Attachment: | 3797-v1.patch added |
---|
Patch to allow setting the color for mythtv's letterboxing.
comment:3 Changed 17 years ago by
Summary: | Patch to add a gray pillarboxes option to Aspect Override → Gray letterboxing option |
---|
I created a new patch based on this concept that allows adding more colour options than just grey. As with Juppers' patch, so far this only works for some X11 based video rendering methods.
comment:4 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
This is a good idea, but I don't like putting the menu and especially as part of aspect override. I'll look at this after the next release.