Skip to content
Snippets Groups Projects
Commit b941da2e authored by George Adrian Stoica's avatar George Adrian Stoica
Browse files

fix more checkstyle warnings

parent 3bf4c2ed
Branches
No related tags found
No related merge requests found
...@@ -161,7 +161,10 @@ public class MetaDataEditorController { ...@@ -161,7 +161,10 @@ public class MetaDataEditorController {
latlong.getMetaData().addTags(((TagsBar.Tag) node).getTag()); latlong.getMetaData().addTags(((TagsBar.Tag) node).getTag());
} }
Event.fireEvent(((Button) event.getSource()).getScene().getWindow(), new MetaDataEvent(METADATA_SAVED)); Event.fireEvent(
((Button) event.getSource()).getScene().getWindow(),
new MetaDataEvent(METADATA_SAVED)
);
closeDialog(event); closeDialog(event);
} }
...@@ -205,7 +208,8 @@ public class MetaDataEditorController { ...@@ -205,7 +208,8 @@ public class MetaDataEditorController {
if (MetaData.STD_PROPERTIES.contains(propertyName)) { if (MetaData.STD_PROPERTIES.contains(propertyName)) {
continue; continue;
} else { } else {
Pair<String, String> p = new Pair<String, String>(propertyName, metaData.getProperty(propertyName)); Pair<String, String> p = new Pair<String, String>(propertyName,
metaData.getProperty(propertyName));
locationProperties.add(p); locationProperties.add(p);
} }
} }
......
...@@ -10,6 +10,8 @@ import javafx.scene.control.TextField; ...@@ -10,6 +10,8 @@ import javafx.scene.control.TextField;
import javafx.scene.input.KeyCode; import javafx.scene.input.KeyCode;
/** /**
* Compunent that allows to use autocomplete with a text field.
*
* @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar * @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar
*/ */
public class AutoCompleteTextField extends TextField { public class AutoCompleteTextField extends TextField {
...@@ -23,7 +25,9 @@ public class AutoCompleteTextField extends TextField { ...@@ -23,7 +25,9 @@ public class AutoCompleteTextField extends TextField {
private StringBuilder sb = new StringBuilder(); private StringBuilder sb = new StringBuilder();
private int lastLength; private int lastLength;
// Constructor /**
* Constructor.
*/
public AutoCompleteTextField() { public AutoCompleteTextField() {
// TextChanged Listener // TextChanged Listener
...@@ -62,7 +66,9 @@ public class AutoCompleteTextField extends TextField { ...@@ -62,7 +66,9 @@ public class AutoCompleteTextField extends TextField {
return entries; return entries;
} }
// TODO ---------------This method not works perfect needs to be redone.... /**
* Todo: This method not works perfect needs to be redone.
*/
public void buggedMethod() { public void buggedMethod() {
// KeyReleased Listener // KeyReleased Listener
setOnKeyReleased(key -> { setOnKeyReleased(key -> {
...@@ -94,7 +100,7 @@ public class AutoCompleteTextField extends TextField { ...@@ -94,7 +100,7 @@ public class AutoCompleteTextField extends TextField {
String originalLowered = getText().toLowerCase(); String originalLowered = getText().toLowerCase();
// Select the first Matching // Select the first Matching
for (String s : entries) for (String s : entries) {
if (s.toLowerCase().startsWith(originalLowered)) { if (s.toLowerCase().startsWith(originalLowered)) {
try { try {
setText(s); setText(s);
...@@ -105,6 +111,7 @@ public class AutoCompleteTextField extends TextField { ...@@ -105,6 +111,7 @@ public class AutoCompleteTextField extends TextField {
selectEnd(); selectEnd();
break; break;
} }
}
}); });
......
...@@ -2,13 +2,14 @@ package simpleex.ui.tags; ...@@ -2,13 +2,14 @@ package simpleex.ui.tags;
import java.util.Arrays; import java.util.Arrays;
import java.util.List; import java.util.List;
import javafx.application.Application; import javafx.application.Application;
import javafx.scene.Scene; import javafx.scene.Scene;
import javafx.scene.layout.VBox; import javafx.scene.layout.VBox;
import javafx.stage.Stage; import javafx.stage.Stage;
/** /**
* Snippet application.
*
* @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar * @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar
*/ */
public class Snippet extends Application { public class Snippet extends Application {
......
package simpleex.ui.tags; package simpleex.ui.tags;
import java.util.SortedSet; import java.util.SortedSet;
import javafx.collections.ObservableList; import javafx.collections.ObservableList;
import javafx.scene.Node; import javafx.scene.Node;
import javafx.scene.control.Label; import javafx.scene.control.Label;
...@@ -16,6 +15,8 @@ import javafx.scene.input.TransferMode; ...@@ -16,6 +15,8 @@ import javafx.scene.input.TransferMode;
import javafx.scene.layout.HBox; import javafx.scene.layout.HBox;
/** /**
* Tag bar component.
*
* @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar * @author GOXR3PLUS https://github.com/goxr3plus/JavaFX-TagsBar
*/ */
public class TagsBar extends HBox { public class TagsBar extends HBox {
...@@ -24,7 +25,9 @@ public class TagsBar extends HBox { ...@@ -24,7 +25,9 @@ public class TagsBar extends HBox {
private ScrollPane scrollPane = new ScrollPane(hBox); private ScrollPane scrollPane = new ScrollPane(hBox);
private AutoCompleteTextField field = new AutoCompleteTextField(); private AutoCompleteTextField field = new AutoCompleteTextField();
// Constructor /**
* Constructor.
*/
public TagsBar() { public TagsBar() {
getStyleClass().setAll("tags-bar"); getStyleClass().setAll("tags-bar");
...@@ -44,8 +47,9 @@ public class TagsBar extends HBox { ...@@ -44,8 +47,9 @@ public class TagsBar extends HBox {
String text = field.getText(); String text = field.getText();
// No Duplicates allowed // No Duplicates allowed
if (!text.isEmpty() && /* getEntries().contains(text) && */ !hBox.getChildren().stream() if (!text.isEmpty() && /* getEntries().contains(text) && */ !hBox.getChildren().stream()
.anyMatch(s -> ((Tag) s).getTag().toLowerCase().equals(text.toLowerCase()))) .anyMatch(s -> ((Tag) s).getTag().toLowerCase().equals(text.toLowerCase()))) {
hBox.getChildren().add(new Tag(text)); hBox.getChildren().add(new Tag(text));
}
field.clear(); field.clear();
}); });
...@@ -53,7 +57,7 @@ public class TagsBar extends HBox { ...@@ -53,7 +57,7 @@ public class TagsBar extends HBox {
} }
/** /**
* Returns all the tags of TagsBar * Returns all the tags of TagsBar.
* *
* @return * @return
*/ */
...@@ -71,7 +75,7 @@ public class TagsBar extends HBox { ...@@ -71,7 +75,7 @@ public class TagsBar extends HBox {
} }
/** /**
* Clears all the tags * Clears all the tags.
* *
*/ */
public void clearAllTags() { public void clearAllTags() {
...@@ -79,9 +83,9 @@ public class TagsBar extends HBox { ...@@ -79,9 +83,9 @@ public class TagsBar extends HBox {
} }
/** /**
* Add this tag if it doesn't exist * Add this tag if it doesn't exist.
* *
* @param tag * @param tag the tag to add
*/ */
public void addTag(String tag) { public void addTag(String tag) {
if (!hBox.getChildren().stream() if (!hBox.getChildren().stream()
...@@ -91,6 +95,8 @@ public class TagsBar extends HBox { ...@@ -91,6 +95,8 @@ public class TagsBar extends HBox {
} }
/** /**
* A tag component.
*
* @author SuperGoliath TagClass * @author SuperGoliath TagClass
*/ */
public class Tag extends HBox { public class Tag extends HBox {
...@@ -98,7 +104,9 @@ public class TagsBar extends HBox { ...@@ -98,7 +104,9 @@ public class TagsBar extends HBox {
private Label textLabel = new Label(); private Label textLabel = new Label();
private Label iconLabel = new Label(null, new ImageView(new Image(getClass().getResourceAsStream("x1.png")))); private Label iconLabel = new Label(null, new ImageView(new Image(getClass().getResourceAsStream("x1.png"))));
// Constructor /**
* Constructor.
*/
public Tag(String tag) { public Tag(String tag) {
getStyleClass().add("tag"); getStyleClass().add("tag");
...@@ -125,8 +133,9 @@ public class TagsBar extends HBox { ...@@ -125,8 +133,9 @@ public class TagsBar extends HBox {
* data is dragged over the target accept it only if it is not dragged from the same imageView and * data is dragged over the target accept it only if it is not dragged from the same imageView and
* if it has a string data * if it has a string data
*/ */
if (event.getGestureSource() != this && event.getDragboard().hasString()) if (event.getGestureSource() != this && event.getDragboard().hasString()) {
event.acceptTransferModes(TransferMode.MOVE); event.acceptTransferModes(TransferMode.MOVE);
}
event.consume(); event.consume();
}); });
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment