Hi,

The attached patch fixes the problem (due to bad logic in the rotation
detection code).

It also reverses the order of the menu entries, making "Normal" the
first entry, followed by "Left", which makes a lot more sense than the
default behavior, IMHO. If you don't want to make this change then don't
apply the second chunk of the patch.

This should probably be sent upstream.

HTH
-- 
René Højbjerg Larsen <[EMAIL PROTECTED]>
--- gnome-randr-applet-0.2/src/grandr.c.orig	2005-12-16 21:14:55.000000000 +0100
+++ gnome-randr-applet-0.2/src/grandr.c	2005-12-16 21:12:00.000000000 +0100
@@ -253,7 +253,7 @@
 
       while (cur != NULL)
 	{
-	  if ((1 << grandr->xr_current_rotation) == ((Rotation) gtk_object_get_data(GTK_OBJECT(cur->data), "rotation_value") & 0xf ))
+	  if ((grandr->xr_current_rotation & 0xf) == (1 << (Rotation) gtk_object_get_data(GTK_OBJECT(cur->data), "rotation_value")))
 	    gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (cur->data), 
 					    TRUE);
 	  cur = g_slist_next(cur);
@@ -346,7 +346,7 @@
       gtk_menu_shell_prepend (GTK_MENU_SHELL(grandr->menu), menu_item);
       gtk_widget_show (menu_item);
 
-      for (i = 0; i < 4; i ++) 
+      for (i = 3; i >= 0; i --) 
 	{
 	  if ((grandr->xr_rotations >> i) & 1)  
 	    {

Reply via email to