On Monday 14 of July 2014 02:49:11 Timothy Pearson wrote:
> > On Sun July 13 2014 7:01:24 pm Timothy Pearson wrote:
> >> I forgot to declare resizeEvent and showEvent as virtual void in
> >> amarok/src/systray.h; I haven't yet tested if fixing this will repair
> >> the
> >> regression.  Aside from this slip-up I don't see anything in that commit
> >> that should affect the menu options whatsoever.
> >>
> >> If you want to try adding the missing virtual keyword to those two
> >> methods
> >> please let me know if it fixes the problem.  Otherwise I'll try to track
> >> this down tomorrow when I have more time.
> >
> > The following patch does not resolve the missing menu:
> >
> > diff -urN amarok/amarok/src/systray.h amarok.new/amarok/src/systray.h
> > --- amarok/amarok/src/systray.h	2014-07-09 19:52:56.000000000 -0500
> > +++ amarok.new/amarok/src/systray.h	2014-07-13 19:26:44.506064803 -0500
> > @@ -30,8 +30,8 @@
> >      virtual void engineTrackPositionChanged( long position, bool
> > /*userSeek*/ );
> >      // get notified of 'highlight' color change
> >      virtual void paletteChange( const TQPalette & oldPalette );
> > -    void resizeEvent ( TQResizeEvent * );
> > -    void showEvent ( TQShowEvent * );
> > +    virtual void resizeEvent ( TQResizeEvent * );
> > +    virtual void showEvent ( TQShowEvent * );
> >
> >  private:
> >      bool event( TQEvent* );
> >
> > Darrell
>
> OK, thanks for testing.
>
> Slavek, can you see any way that this commit could have affected the tray
> icon menu?
> https://git.trinitydesktop.org/cgit/amarok/commit/?id=f3088bc3de7db3d3bf523
>4cb8327b2bfa1e0a025
>
> Thanks!
>
> Tim
>

Yes, I can look at it, but later - now I have time to go to sleep :)

-- 
Slavek