[brailleblaster] push by mwhapples - Put the radiobuttons in a group to allow tab access. There is still a ... on 2012-07-26 00:02 GMT

  • From: brailleblaster@xxxxxxxxxxxxxx
  • To: brailleblaster@xxxxxxxxxxxxx
  • Date: Thu, 26 Jul 2012 00:02:53 +0000

Revision: d43f64d1f187
Author:   Michael Whapples
Date:     Wed Jul 25 16:58:21 2012
Log: Put the radiobuttons in a group to allow tab access. There is still a bug where cycling round (between cancel and the radiobuttons) will take focus to the Shell.
http://code.google.com/p/brailleblaster/source/detail?r=d43f64d1f187

Modified:
 /src/main/org/brailleblaster/wordprocessor/DocumentManager.java

=======================================
--- /src/main/org/brailleblaster/wordprocessor/DocumentManager.java Tue Jul 24 22:12:13 2012 +++ /src/main/org/brailleblaster/wordprocessor/DocumentManager.java Wed Jul 25 16:58:21 2012
@@ -694,21 +694,22 @@
         FillLayout layout = new FillLayout(SWT.VERTICAL);
         layout.marginWidth = 8;
         selShell.setLayout(layout);
-
-        final Button b1 = new Button (selShell, SWT.RADIO);
+        final Composite radioGroup = new Composite(selShell, SWT.NONE);
+        radioGroup.setLayout(new RowLayout(SWT.VERTICAL));
+        final Button b1 = new Button (radioGroup, SWT.RADIO);
                b1.setText(lh.localValue("saveTextBraille"));
                if (braille.view.getCharCount() == 0) {
                        b1.setEnabled(false);
                }
                b1.pack();

-               final Button b2 = new Button (selShell, SWT.RADIO);
+               final Button b2 = new Button (radioGroup, SWT.RADIO);
                b2.setText(lh.localValue("saveTextOnly"));
                if (braille.view.getCharCount() == 0) {
                        b2.setSelection(true);
                }
                b2.pack();
-
+               radioGroup.pack();
                Composite c = new Composite(selShell, SWT.NONE);
                RowLayout clayout = new RowLayout();
                clayout.type = SWT.HORIZONTAL;

Other related posts:

  • » [brailleblaster] push by mwhapples - Put the radiobuttons in a group to allow tab access. There is still a ... on 2012-07-26 00:02 GMT - brailleblaster