summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjan.nijtmans <nijtmans@users.sourceforge.net>2012-05-04 22:18:56 (GMT)
committerjan.nijtmans <nijtmans@users.sourceforge.net>2012-05-04 22:18:56 (GMT)
commit5161961e105852e93b7331def1f5868d79bcfeb8 (patch)
tree406d8c0393b643f248a743b868ba345f4a31882c
parent8db6c7104c4eab7d08aa77859aff497152279fd7 (diff)
parentd1a95d8859199b3fff57d349db42d70b1feee84c (diff)
downloadtk-5161961e105852e93b7331def1f5868d79bcfeb8.zip
tk-5161961e105852e93b7331def1f5868d79bcfeb8.tar.gz
tk-5161961e105852e93b7331def1f5868d79bcfeb8.tar.bz2
[Bug 2768586]: Menu posting problem on dual monitors
-rw-r--r--ChangeLog4
-rw-r--r--library/menu.tcl17
2 files changed, 12 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index f36f8cc..922770f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2012-05-04 Jan Nijtmans <nijtmans@users.sf.net>
+
+ * library/menu.tcl: [Bug 2768586]: Menu posting problem on dual monitors
+
2012-05-02 Jan Nijtmans <nijtmans@users.sf.net>
* library/tk.tcl: [Bug 533519]: Window placement with multiple screens
diff --git a/library/menu.tcl b/library/menu.tcl
index e00dad9..8337eae 100644
--- a/library/menu.tcl
+++ b/library/menu.tcl
@@ -286,8 +286,8 @@ proc ::tk::MbPost {w {x {}} {y {}}} {
set x [winfo rootx $w]
set y [expr {[winfo rooty $w] - [winfo reqheight $menu]}]
# if we go offscreen to the top, show as 'below'
- if {$y < 0} {
- set y [expr {[winfo rooty $w] + [winfo height $w]}]
+ if {$y < [winfo vrooty $w]} {
+ set y [expr {[winfo vrooty $w] + [winfo rooty $w] + [winfo reqheight $w]}]
}
PostOverPoint $menu $x $y
}
@@ -296,8 +296,8 @@ proc ::tk::MbPost {w {x {}} {y {}}} {
set y [expr {[winfo rooty $w] + [winfo height $w]}]
# if we go offscreen to the bottom, show as 'above'
set mh [winfo reqheight $menu]
- if {($y + $mh) > [winfo screenheight $w]} {
- set y [expr {[winfo rooty $w] - $mh}]
+ if {($y + $mh) > ([winfo vrooty $w] + [winfo vrootheight $w])} {
+ set y [expr {[winfo vrooty $w] + [winfo vrootheight $w] + [winfo rooty $w] - $mh}]
}
PostOverPoint $menu $x $y
}
@@ -1240,16 +1240,15 @@ proc ::tk::PostOverPoint {menu x y {entry {}}} {
if {$ver < 6} {
set yoffset [expr {[winfo screenheight $menu] \
- $y - [winfo reqheight $menu] - 10}]
- if {$yoffset < 0} {
+ if {$yoffset < [winfo vrooty $menu]} {
# The bottom of the menu is offscreen, so adjust upwards
- incr y $yoffset
- if {$y < 0} { set y 0 }
+ incr y [expr {$yoffset - [winfo vrooty $menu]}]
}
# If we're off the top of the screen (either because we were
# originally or because we just adjusted too far upwards),
# then make the menu popup on the top edge.
- if {$y < 0} {
- set y 0
+ if {$y < [winfo vrooty $menu]} {
+ set y [winfo vrooty $menu]
}
}
}