[mythtv] [mythtv-commits] mythtv/master commit: 9b7eae2fd by Jonatan Lindblad (natanojl)
Jonatan Lindblad
mythtv at comhem.se
Sun Feb 19 22:44:45 UTC 2012
2012-02-19 17:24, Paul Harrison skrev:
> On 19/02/12 15:57, Jonatan Lindblad wrote:
>> 2012-02-18 18:49, Paul Harrison skrev:
>>> On 18/02/12 14:19, Jonatan Lindblad wrote:
>>>> 2012-02-18 14:52, Paul Harrison skrev:
>>>>> On 12/02/12 23:50, MythTV wrote:
>>>>>> Author: Jonatan Lindblad<jlindblad at mythtv.org>
>>>>>> Change Date: 2012-02-12T15:15:15-08:00
>>>>>> Push Date: 2012/02/12 15:49:53 -0800
>>>>>> Repository: mythtv
>>>>>> Branch: master
>>>>>> New Revision: 9b7eae2fda2e15d20bc846dcef756c05f09d6ac0
>>>>>> Changeset: https://github.com/MythTV/mythtv/commit/9b7eae2fd
>>>>>>
>>>>>> Log:
>>>>>>
>>>>>> MythUI: Add a scroll bar widget.
>>>>>>
>>>>>> Add a scroll bar widget and use it in MythUIButtonList.
>>>>>>
>>>>> Hi Jonatan,
>>>>>
>>>>> This seems to be causing images in the buttonlists to sometimes not be
>>>>> updated properly.
>>>>>
>>>>> The place I noticed it is in MythMusic if you replace the tracks in
>>>>> the
>>>>> current playlist (eq if you select one of the quick playlist options)
>>>>> the coverart images don't update until you scroll in the list.
>>>>>
>>>>> Paul H.
>>>> Hi Paul,
>>>>
>>>> I have reproduced the problem and will try to fix it.
>>>>
>>>> --
>>>> Jonatan
>>>
>>> OK thanks for looking at it. I suspect it's the changes to the
>>> m_needsUpdate flag which is always set to false in DrawSelf() now were
>>> before it wasn't if m_clearing was true because SetPositionArrowStates()
>>> bailed out early before it set m_needsUpdate to false. That would
>>> explain why it only happens after the buttonlist is reset.
>>
>> Yes, that was it. Should be fixed with d95479d83b. From what I could
>> see it only happened when setting images triggered by the itemVisible
>> signal.
>>
>
> Verified fixed thanks.
>
>>
>>> I'm also seeing strange behaviour in MythBrowser when scrolling using
>>> the mouse. I haven't reverted to see if it definitely is your other
>>> scrollbar widget commit but I can't think of anything else that may have
>>> broken it.
>>>
>>> Paul H.
>>
>> Can you try the attached patch and see if that works?
>>
>> --
>> Jonatan
>
>
> That seems to fix it. The only odd thing I noticed is using the keyboard
> or scroll wheel you can't scroll all the way down or right.
>
> Paul H.
I just pushed 1ed8ec2 which I think should fix these issues.
--
Jonatan
More information about the mythtv-dev
mailing list