From bf316d4202a0aa6e351cf6d37b08d5cd0b7da00b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marko=20Gr=C3=B6nroos?= Date: Thu, 18 Nov 2010 15:47:44 +0000 Subject: [PATCH] Reverted [16045] because of invalid formatting. svn changeset:16046/svn branch:6.5 --- .../terminal/gwt/client/ui/VTextField.java | 37 +++-------- src/com/vaadin/ui/TextField.java | 8 +-- .../com/vaadin/tests/tickets/Ticket6003.java | 61 ------------------- 3 files changed, 14 insertions(+), 92 deletions(-) delete mode 100644 tests/src/com/vaadin/tests/tickets/Ticket6003.java diff --git a/src/com/vaadin/terminal/gwt/client/ui/VTextField.java b/src/com/vaadin/terminal/gwt/client/ui/VTextField.java index 4ba6256b29..2bfa3f5cdc 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VTextField.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VTextField.java @@ -67,7 +67,6 @@ public class VTextField extends TextBoxBase implements Paintable, Field, private String inputPrompt = null; private boolean prompting = false; private int lastCursorPos = -1; - private boolean wordwrap = true; public VTextField() { this(DOM.createInputText()); @@ -109,7 +108,8 @@ public class VTextField extends TextBoxBase implements Paintable, Field, } if (listenTextChangeEvents - && (event.getTypeInt() & TEXTCHANGE_EVENTS) == event.getTypeInt()) { + && (event.getTypeInt() & TEXTCHANGE_EVENTS) == event + .getTypeInt()) { deferTextChangeEvent(); } @@ -199,18 +199,20 @@ public class VTextField extends TextBoxBase implements Paintable, Field, inputPrompt = uidl.getStringAttribute(ATTR_INPUTPROMPT); - setMaxLength(uidl.hasAttribute("maxLength") ? uidl.getIntAttribute("maxLength") - : -1); + setMaxLength(uidl.hasAttribute("maxLength") ? uidl + .getIntAttribute("maxLength") : -1); immediate = uidl.getBooleanAttribute("immediate"); listenTextChangeEvents = client.hasEventListeners(this, "ie"); if (listenTextChangeEvents) { - textChangeEventMode = uidl.getStringAttribute(ATTR_TEXTCHANGE_EVENTMODE); + textChangeEventMode = uidl + .getStringAttribute(ATTR_TEXTCHANGE_EVENTMODE); if (textChangeEventMode.equals(TEXTCHANGE_MODE_EAGER)) { textChangeEventTimeout = 1; } else { - textChangeEventTimeout = uidl.getIntAttribute(ATTR_TEXTCHANGE_TIMEOUT); + textChangeEventTimeout = uidl + .getIntAttribute(ATTR_TEXTCHANGE_TIMEOUT); } sinkEvents(TEXTCHANGE_EVENTS); attachCutEventListener(getElement()); @@ -220,8 +222,8 @@ public class VTextField extends TextBoxBase implements Paintable, Field, setColumns(new Integer(uidl.getStringAttribute("cols")).intValue()); } - final String text = uidl.hasVariable("text") ? uidl.getStringVariable("text") - : null; + final String text = uidl.hasVariable("text") ? uidl + .getStringVariable("text") : null; setPrompting(inputPrompt != null && focusedTextField != this && (text == null || text.equals(""))); @@ -278,11 +280,6 @@ public class VTextField extends TextBoxBase implements Paintable, Field, } }); } - - // For backward compatibility; to be moved to TextArea - if (uidl.hasAttribute("wordwrap")) { - setWordwrap(uidl.getBooleanAttribute("wordwrap")); - } } protected void onCut() { @@ -541,18 +538,4 @@ public class VTextField extends TextBoxBase implements Paintable, Field, valueChange(false); } - // For backward compatibility; to be moved to TextArea - public void setWordwrap(boolean enabled) { - if (enabled == wordwrap) - return; // No change - - if (enabled) { - DOM.removeElementAttribute(getElement(), "wrap"); - DOM.setStyleAttribute(getElement(), "overflow", null); - } else { - DOM.setElementAttribute(getElement(), "wrap", "off"); - DOM.setStyleAttribute(getElement(), "overflow", "auto"); - } - wordwrap = enabled; - } } diff --git a/src/com/vaadin/ui/TextField.java b/src/com/vaadin/ui/TextField.java index e4a48f52de..245c498959 100644 --- a/src/com/vaadin/ui/TextField.java +++ b/src/com/vaadin/ui/TextField.java @@ -198,9 +198,9 @@ public class TextField extends AbstractTextField implements if (rows != 0) { target.addAttribute("rows", String.valueOf(rows)); target.addAttribute("multiline", true); - - // For backward compatibility; to be moved to TextArea - target.addAttribute("wordwrap", isWordwrap()); + if (!isWordwrap()) { + target.addAttribute("wordwrap", false); + } } if (getInputPrompt() != null) { @@ -484,7 +484,7 @@ public class TextField extends AbstractTextField implements textChangeEventPending = true; } - /* ** Text Change Events ** */ + /* ** Text Change Events ** */ /** * Sets the mode how the TextField triggers {@link TextChangeEvent}s. diff --git a/tests/src/com/vaadin/tests/tickets/Ticket6003.java b/tests/src/com/vaadin/tests/tickets/Ticket6003.java deleted file mode 100644 index 4e3f129a6a..0000000000 --- a/tests/src/com/vaadin/tests/tickets/Ticket6003.java +++ /dev/null @@ -1,61 +0,0 @@ -package com.vaadin.tests.tickets; - -import com.vaadin.data.Property; -import com.vaadin.data.Property.ValueChangeEvent; -import com.vaadin.tests.components.TestBase; -import com.vaadin.ui.CheckBox; -import com.vaadin.ui.HorizontalLayout; -import com.vaadin.ui.TextArea; -import com.vaadin.ui.VerticalLayout; -import com.vaadin.ui.Window; - -public class Ticket6003 extends TestBase { - - @Override - public void setup() { - Window main = new Window("The Main Window"); - setMainWindow(main); - - final VerticalLayout mainLayout = new VerticalLayout(); - main.setContent(mainLayout); - - HorizontalLayout layout = new HorizontalLayout(); - - TextArea area1 = new TextArea("Wrapping"); - area1.setWordwrap(true); // The default - area1.setValue("A quick brown fox jumps over the lazy dog"); - - final TextArea area2 = new TextArea("Nonwrapping"); - area2.setWordwrap(false); - area2.setValue("Victor jagt zwölf Boxkämpfer quer " - + "über den Sylter Deich"); - - layout.addComponent(area1); - layout.addComponent(area2); - layout.setSpacing(true); - - mainLayout.addComponent(layout); - - CheckBox onoff = new CheckBox("Wrap state for the right field"); - onoff.setValue(false); - onoff.addListener(new Property.ValueChangeListener() { - @Override - public void valueChange(ValueChangeEvent event) { - area2.setWordwrap((Boolean) event.getProperty().getValue()); - } - }); - onoff.setImmediate(true); - - mainLayout.addComponent(onoff); - } - - @Override - protected String getDescription() { - return ""; - } - - @Override - protected Integer getTicketNumber() { - return 6003; - } -} -- 2.39.5