Changeset 1768 for branches/eraser6/CodeReview/Eraser/SchedulerPanel.cs

Timestamp:

2/8/2010 7:20:53 AM (2 years ago)

Author:

lowjoel

Message:

Also specify MessageBoxOptions?.RightAlign? if the dialog which owns the message box is RTL-reading. addresses #275: Code Review

File:

• branches/eraser6/CodeReview/Eraser/SchedulerPanel.cs (modified) (2 diffs)

branches/eraser6/CodeReview/Eraser/SchedulerPanel.cs

@@ -146 +146 @@
         {
             if (MessageBox.Show(this, S._("Are you sure you want to delete the selected tasks?"),
-                   S._("Eraser"), MessageBoxButtons.YesNo, MessageBoxIcon.Question,
-                   MessageBoxDefaultButton.Button1,
-                   S.IsRightToLeft(this) ? MessageBoxOptions.RtlReading : 0) != DialogResult.Yes)
+                    S._("Eraser"), MessageBoxButtons.YesNo, MessageBoxIcon.Question,
+                    MessageBoxDefaultButton.Button1, S.IsRightToLeft(this) ?
+                        MessageBoxOptions.RtlReading | MessageBoxOptions.RightAlign : 0
+                ) != DialogResult.Yes)
             {
                 return;
@@ -445 +446 @@
                             catch (SerializationException ex)
                             {
-                                MessageBox.Show(S._("Could not import task list from {0}. The error " +
-                                    "returned was: {1}", file, ex.Message), S._("Eraser"),
+                                MessageBox.Show(S._("Could not import task list from {0}. The " +
+                                    "error returned was: {1}", file, ex.Message), S._("Eraser"),
                                     MessageBoxButtons.OK, MessageBoxIcon.Error,
                                     MessageBoxDefaultButton.Button1,
-                                    S.IsRightToLeft(null) ? MessageBoxOptions.RtlReading : 0);
+                                    S.IsRightToLeft(null) ?
+                                        MessageBoxOptions.RtlReading | MessageBoxOptions.RightAlign : 0);
                             }
                         }