[[ Bug 13675 ]] Scrollbar for the font selection in the script editor preferences doesn't work

livecodesebastien · livecodesebastien · commit 1fb12d17ba0a · 2014-10-17T16:45:57.000+01:00
diff --git a/docs/notes/bugfix-13675.md b/docs/notes/bugfix-13675.md
@@ -0,0 +1 @@
+# Scrollbar for the font selection in the script editor preferences doesn't work
diff --git a/engine/src/button.cpp b/engine/src/button.cpp
@@ -1136,8 +1136,17 @@ Boolean MCButton::mdown(uint2 which)
 	{
         // SN-2014-08-26: [[ Bug 13201 ]] mx/my are now related to the button's rectangle,
         //  not the stack's rectangle anymore.
-		if (state & CS_SCROLLBAR && mx > rect.width - MCscrollbarwidth
-		        && mx < rect.width)
+        // SN-2014-10-17: [[ Bug 13675 ]] mx/my refer to the button's rectangle on Mac only
+        int2 t_right_limit, t_left_limit;
+#ifdef _MACOSX
+        t_left_limit = rect.width - MCscrollbarwidth;
+        t_right_limit = rect.width;
+#else
+        t_left_limit = rect.x + rect.width - MCscrollbarwidth;
+        t_right_limit = rect.x + rect.width;
+#endif
+        if (state & CS_SCROLLBAR && mx > t_left_limit
+                && mx < t_right_limit)
 		{
 			menu->mdown(which);
 			state |= CS_FIELD_GRAB;
@@ -4723,4 +4732,4 @@ IO_stat MCButton::load(IO_handle stream, const char *version)
 		}
 	}
 	return IO_NORMAL;
-}	
+}

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+# Scrollbar for the font selection in the script editor preferences doesn't work`