Modify

Ticket #9674 (closed Bug Report - General: fixed)

Opened 2 years ago

Last modified 2 years ago

MythUIEditBar broken

Reported by: Jim Stichnoth <stichnot@…> Owned by: stuartm
Priority: minor Milestone: 0.25
Component: MythTV - User Interface Library Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by wagnerrp) (diff)

Some fairly recent change in trunk has broken the MythUIEditBar when editing a recording. It appears that the "keep" and "cut" shapes are not being cropped to the correct width. My best guess is that this started with [e949f5fe9d9dd267bdeedd1a8de4588c43a1d8ce].

This happens with all themes I tried (Blue Abstract, MythBuntu?, and Terra), and with Slim and VDPAU Normal playback profiles.

Same behavior when unapplying my patches and using a clean build.

Attachments

9674.patch (1.4 KB) - added by Jim Stichnoth <stichnot@…> 2 years ago.
9674.2.patch (1.4 KB) - added by Jim Stichnoth <stichnot@…> 2 years ago.
version_info (727 bytes) - added by wagnerrp 2 years ago.

Change History

comment:1 Changed 2 years ago by markk

I've just tested on two boxes with the latest master on both and I can't see any problems. If there were issues with the latest painter changes, they would be showing up elsewhere and not just in the edit bar. Are you sure you're up to date? - there was a fix that went in for the shape cache.

Changed 2 years ago by Jim Stichnoth <stichnot@…>

comment:2 Changed 2 years ago by Jim Stichnoth <stichnot@…>

The attached patch fixes the problem I'm seeing.

Changed 2 years ago by Jim Stichnoth <stichnot@…>

comment:3 Changed 2 years ago by Jim Stichnoth <stichnot@…>

Uploaded a slightly cleaner / more efficient version of the patch.

To address Mark's question, the problem should show up on the edit bar for any theme in which the "editbar" section of osd.xml has "keep" defined as a shape (as in default-wide or Terra), rather than as an imagetype (as in MythCenter?-wide). The width of every "keep" bar becomes the full length of the recording, instead of being cropped to the correct duration, effectively hiding all "cut" bars except for the first one. There are other incorrect visual artifacts, but this is the most glaring one. Also, it's not surprising that the problem doesn't show up widely, since MythUIEditBar and MythUIProgressBar are the only internal components that use cropping on shapes.

comment:4 Changed 2 years ago by wagnerrp

  • Description modified (diff)

Changed 2 years ago by wagnerrp

comment:5 Changed 2 years ago by Jim Stichnoth

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from unknown to 0.25

libmythui: Fix rendering of MythUIEditBar

Some cropping adjustments were lost when the painter classes where re-factored and MythUIShape rendering was moved into the painters.

Closes #9674.

Signed-off-by: Mark Kendall <mkendall@…>

Changeset: 95a9b5df134eca166a111512c3d53d8cc0e20e62

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.