Bug 444711

Summary: popup menu overlaps to the panel
Product: [Fedora] Fedora Reporter: Akira TAGOH <tagoh>
Component: scimAssignee: Peng Huang <phuang>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: 9CC: i18n-bugs, petersen
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-01-29 07:55:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch to fix it.
none
patch without the global variable none

Description Akira TAGOH 2008-04-30 07:55:48 UTC 
Description of problem:
When the systray is sitting down at the top panel, the popup menu appearing by
clicking the mouse button is put at where you clicked. but others are aligned to
the panel and doesn't overlap. this behavior sometimes mistakenly hits the item
unintentionally.

Version-Release number of selected component (if applicable):
scim-1.4.7-21.fc9
Comment 1 Bug Zapper 2008-05-14 10:25:12 UTC 
Changing version to '9' as part of upcoming Fedora 9 GA.
More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Comment 2 Akira TAGOH 2008-07-17 02:30:51 UTC 
Easier way to accomplish this would be to use GtkStatusIcon instead of using the
code based on eggtrayicon.
Comment 3 Peng Huang 2008-07-17 06:13:00 UTC 
SCIM panel popup the menu is in panel_agent's slot. It can not get the location
information easily. It needs a lot of modification. And consider the severity is
low, so I suggest wontfix.
Comment 4 Peng Huang 2008-07-17 08:51:43 UTC 
Created attachment 312015 [details]
patch to fix it.

This patch fixes this problem via adding several global variables.
Comment 5 Peng Huang 2008-07-17 09:10:55 UTC 
Fixed in scim-1_4_7-28_fc10
Comment 6 Akira TAGOH 2008-07-17 09:27:42 UTC 
Created attachment 312020 [details]
patch without the global variable

(In reply to comment #4)
> Created an attachment (id=312015) [edit]
> patch to fix it.
> 
> This patch fixes this problem via adding several global variables. 

If you don't want to use the global variables - I haven't tested this but
should works.
Comment 7 Jens Petersen 2008-07-23 08:11:08 UTC 
Thanks!  28.fc10 looks good. :-)

You may still want to consider Tagoh's suggestion.
Comment 8 Tony Fu 2008-09-10 03:17:41 UTC 
requested by Jens Petersen (#27995)
Comment 9 Jens Petersen 2008-12-02 00:46:04 UTC 
Huang Peng, do you want to update the patch to comment 6?
Comment 10 Peng Huang 2008-12-02 04:11:24 UTC 
I think both it and mine are OK. But I did not have time to test Tagoh's patch.