Fixes #532 by adding a Tab to open channels

FAForever · Jun 11, 2017 · 36caa2d · 36caa2d
1 parent 8d91b85
commit 36caa2d
Show file tree

Hide file tree

Showing 4 changed files with 69 additions and 27 deletions.
diff --git a/src/main/java/com/faforever/client/chat/ChatController.java b/src/main/java/com/faforever/client/chat/ChatController.java
@@ -11,7 +11,6 @@
 import com.google.common.eventbus.EventBus;
 import com.google.common.eventbus.Subscribe;
 import javafx.application.Platform;
-import javafx.beans.InvalidationListener;
 import javafx.collections.ListChangeListener;
 import javafx.scene.Node;
 import javafx.scene.control.Tab;
@@ -41,8 +40,9 @@ public class ChatController extends AbstractViewController<Node> {
   public Node chatRoot;
   public TabPane tabPane;
   public Pane connectingProgressPane;
-  public VBox noOpenTabsContainer;
+  public VBox newChannelJoinRoot;
   public TextField channelNameTextField;
+  private Tab newChannel;
 
   @Inject
   public ChatController(ChatService chatService, UiService uiService, UserService userService, EventBus eventBus) {
@@ -65,19 +65,16 @@ private void onChannelJoined(Channel channel) {
   private void onDisconnected() {
     connectingProgressPane.setVisible(true);
     tabPane.setVisible(false);
-    noOpenTabsContainer.setVisible(false);
   }
 
   private void onConnected() {
     connectingProgressPane.setVisible(false);
     tabPane.setVisible(true);
-    noOpenTabsContainer.setVisible(false);
   }
 
   private void onConnecting() {
     connectingProgressPane.setVisible(true);
     tabPane.setVisible(false);
-    noOpenTabsContainer.setVisible(false);
   }
 
   private void onLoggedOut() {
@@ -106,15 +103,31 @@ private void addTab(String playerOrChannelName, AbstractChatTabController tabCon
     JavaFxUtil.assertApplicationThread();
     nameToChatTabController.put(playerOrChannelName, tabController);
     tabPane.getTabs().add(tabController.getRoot());
+    tabPane.getTabs().sort((o1, o2) -> {
+      if (o1.equals(newChannel)) {
+        return 1;
+      } else if (o2.equals(newChannel)) {
+        return -1;
+      } else {
+        return 0;
+      }
+    });
+    tabPane.getSelectionModel().select(tabController.getRoot());
   }
 
   @Override
   public void initialize() {
     super.initialize();
     eventBus.register(this);
 
-    tabPane.getTabs().addListener((InvalidationListener) observable ->
-        noOpenTabsContainer.setVisible(tabPane.getTabs().isEmpty()));
+    NewChannelController newChannelController = uiService.loadFxml("theme/chat/newChannel.fxml");
+    newChannelJoinRoot = (VBox) newChannelController.getRoot();
+    channelNameTextField = newChannelController.getChannelNameTextField();
+    channelNameTextField.setOnAction(event -> onJoinChannelButtonClicked());
+
+    newChannel = new Tab("+", newChannelJoinRoot);
+    newChannel.setClosable(false);
+    tabPane.getTabs().add(newChannel);
 
     chatService.addChannelsListener(change -> {
       if (change.wasRemoved()) {
@@ -240,7 +253,7 @@ private boolean isCurrentUser(ChatUser chatUser) {
   @Override
   protected void onDisplay() {
     super.onDisplay();
-    if (!tabPane.getTabs().isEmpty()) {
+    if (tabPane.getTabs().size() > 1) {
       Tab tab = tabPane.getSelectionModel().getSelectedItem();
       nameToChatTabController.get(tab.getId()).onDisplay();
     }

diff --git a/src/main/java/com/faforever/client/chat/NewChannelController.java b/src/main/java/com/faforever/client/chat/NewChannelController.java
@@ -0,0 +1,28 @@
+package com.faforever.client.chat;
+
+
+import com.faforever.client.fx.Controller;
+import javafx.fxml.FXML;
+import javafx.scene.control.TextField;
+import javafx.scene.layout.VBox;
+import org.springframework.beans.factory.config.ConfigurableBeanFactory;
+import org.springframework.context.annotation.Scope;
+import org.springframework.stereotype.Component;
+
+@Component
+@Scope(ConfigurableBeanFactory.SCOPE_SINGLETON)
+public class NewChannelController implements Controller {
+  @FXML
+  public VBox root;
+  @FXML
+  public TextField channelNameTextField;
+
+  @Override
+  public Object getRoot() {
+    return root;
+  }
+
+  public TextField getChannelNameTextField() {
+    return channelNameTextField;
+  }
+}
diff --git a/src/main/resources/theme/chat/chat.fxml b/src/main/resources/theme/chat/chat.fxml
@@ -3,9 +3,7 @@
 <?import javafx.geometry.Insets?>
 <?import javafx.scene.control.Label?>
 <?import javafx.scene.control.ProgressIndicator?>
-<?import javafx.scene.control.Separator?>
 <?import javafx.scene.control.TabPane?>
-<?import javafx.scene.control.TextField?>
 <?import javafx.scene.layout.AnchorPane?>
 <?import javafx.scene.layout.VBox?>
 <AnchorPane xmlns:fx="http://javafx.com/fxml/1" fx:id="chatRoot" xmlns="http://javafx.com/javafx/8.0.40"
@@ -25,22 +23,6 @@
         </Label>
       </children>
     </VBox>
-    <VBox fx:id="noOpenTabsContainer" alignment="CENTER" fillWidth="false" AnchorPane.bottomAnchor="0.0"
-          AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
-      <children>
-        <VBox>
-          <children>
-            <Label styleClass="h2" text="%chat.noOpenChats"/>
-            <Separator prefWidth="300.0"/>
-            <Label text="%chat.joinAChannel">
-              <padding>
-                <Insets top="10.0"/>
-              </padding>
-            </Label>
-            <TextField fx:id="channelNameTextField" onAction="#onJoinChannelButtonClicked" promptText="%chat.channelNamePrompt"/>
-          </children>
-        </VBox>
-      </children>
-    </VBox>
+
   </children>
 </AnchorPane>
diff --git a/src/main/resources/theme/chat/newChannel.fxml b/src/main/resources/theme/chat/newChannel.fxml
@@ -0,0 +1,19 @@
+<?import javafx.geometry.Insets?>
+<?import javafx.scene.control.Label?>
+<?import javafx.scene.control.TextField?>
+<?import javafx.scene.layout.VBox?>
+<VBox xmlns:fx="http://javafx.com/fxml/1" fx:controller="com.faforever.client.chat.NewChannelController" fx:id="root"
+      alignment="CENTER" fillWidth="false">
+    <children>
+        <VBox>
+            <children>
+                <Label text="%chat.joinAChannel">
+                    <padding>
+                        <Insets top="10.0"/>
+                    </padding>
+                </Label>
+                <TextField fx:id="channelNameTextField" promptText="%chat.channelNamePrompt"/>
+            </children>
+        </VBox>
+    </children>
+</VBox>