Bug 245178

Description Rob Clevenger 2008-08-25 17:36:24 EDT

Created attachment 110847 [details]
7711 characters

Build ID: I20080617-2000

Steps To Reproduce:
I've seen this in Eclipse dialogs (mainly the BuildPathsBlock listing out the included and excluded nodes in the tree).  When you have too much text in a SWT control (7710 seems to be the magic number), it falls over and wraps on top of itself causing garbled text.

I made a simple Eclipse command handler with this code to demonstrate the problem.  update 7711 with some larger value to see the text get more garbled (as it is, only the 1st character looks off with 7711 characters since 7710 seems to be the limit where this starts wrapping.  This is NOT limited to Text components though, since I've seen in the in org.eclipse.jdt.internal.ui.wizards.buildpaths.CPListLabelProvider

More information:
package foo.handlers;

import org.eclipse.core.commands.AbstractHandler;
import org.eclipse.core.commands.ExecutionEvent;
import org.eclipse.core.commands.ExecutionException;
import org.eclipse.swt.SWT;
import org.eclipse.swt.layout.GridData;
import org.eclipse.swt.layout.GridLayout;
import org.eclipse.swt.widgets.Composite;
import org.eclipse.swt.widgets.Control;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.Shell;
import org.eclipse.swt.widgets.Text;
import org.eclipse.ui.IWorkbenchWindow;
import org.eclipse.ui.handlers.HandlerUtil;
import org.eclipse.jface.dialogs.Dialog;

/**
 * Our sample handler extends AbstractHandler, an IHandler base class.
 * @see org.eclipse.core.commands.IHandler
 * @see org.eclipse.core.commands.AbstractHandler
 */
public class SampleHandler extends AbstractHandler {
	/**
	 * The constructor.
	 */
	public SampleHandler() {
	}

	/**
	 * the command has been executed, so extract extract the needed information
	 * from the application context.
	 */
	public Object execute(ExecutionEvent event) throws ExecutionException {
		IWorkbenchWindow window = HandlerUtil.getActiveWorkbenchWindowChecked(event);
		MyDialog d = new MyDialog(window.getShell());
		d.open();
		return null;
	}

	static class MyDialog extends Dialog {
      protected MyDialog(Shell parentShell) {
        super(parentShell);
      }

      @Override
      protected Control createDialogArea(Composite parent) {
        StringBuilder s = new StringBuilder(100* 1000);
        for (int i=0; i < 7711; i++) {
          s.append((char) ((i % 26) + 65));
        }

        Composite c = (Composite) super.createDialogArea(parent);
        c.setLayout(new GridLayout(1, false));
        new Label(c, SWT.CENTER).setText("Hello World");

        Label l = new Label(c, SWT.NONE);
        l.setText(s.toString());
        l.setLayoutData(new GridData(SWT.FILL, SWT.TOP, true, false, 1, 1));

        Text t = new Text(c, SWT.BORDER |SWT.SINGLE);
        t.setText(s.toString());
        t.setLayoutData(new GridData(SWT.FILL, SWT.TOP, true, false, 1, 1));
        this.getShell().setSize(1010, 400);
        return c;
      }

      @Override
      public int open() {
        return super.open();
      }

	}
}