Bug 278365

Summary:	ContextMenu side image not visible anymore
Product:	[Applications] digikam	Reporter:	Andi Clemens <andi.clemens>
Component:	Usability-Menus	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	minor	CC:	caulier.gilles, cfeck
Priority:	NOR
Version:	2.0.0
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:	2.5.0
Sentry Crash Report:

Description Andi Clemens 2011-07-23 22:47:35 UTC

Version:           2.0.0 (using KDE 4.6.4) 
OS:                Linux

The context menu had a side image before, now it seems to be gone.
Has it been removed? If so, we should fix the width of the context menu, because when viewing an image and right-clicking on it, the context menu is wider than in the thumbnail view.

Reproducible: Always

Steps to Reproduce:
1. Click on an thumbnail.
2. Right click on the image

Actual Results:  
Context menu shows no side image, context menu is too wide.

Expected Results:  
The image should be displayed. If we removed it, the context menu width needs to be fixed.

OS: Linux (i686) release 2.6.39-ARCH
Compiler: gcc

Comment 1 Christoph Feck 2011-07-24 01:44:06 UTC

Yes, a regression from bug 274388. Actually, I note that possible issue, but you are so fast with bug fixing :)

There are three things that need to be rendered:
- the background/frame (PE_FrameMenu)
- the icons/text (actual QMenu)
- the sidebar (your customization)

The rendering order of those elements should match the list above.

Comment 2 Christoph Feck 2011-07-24 01:50:46 UTC

If the intention was to put the frame above the sidebar, then you might need to explicitely check for QWindowsVistaStyle, and only apply the new rendering order for that. Use style()->inherits().

Comment 3 caulier.gilles 2011-12-13 10:10:20 UTC

context menu image have been removed definitively with 2.3.0 release...

Gilles Caulier