Changed menu UI

src/main/kotlin/com/jetpackduba/gitnuro/exceptions/RevertCommitException.kt

@@ -0,0 +1,4 @@
+package com.jetpackduba.gitnuro.exceptions
+
+class RevertCommitException {
+}

src/main/kotlin/com/jetpackduba/gitnuro/ui/Menu.kt

@@ -8,8 +8,6 @@ import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.*
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material.*
-import androidx.compose.material.icons.Icons
-import androidx.compose.material.icons.filled.ArrowDropDown
 import androidx.compose.runtime.*
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.ExperimentalComposeUiApi
@@ -33,9 +31,10 @@ fun Menu(
     modifier: Modifier,
     menuViewModel: MenuViewModel = gitnuroViewModel(),
     onCreateBranch: () -> Unit,
-    onGoToWorkspace: () -> Unit,
+    onOpenAnotherRepository: () -> Unit,
     onStashWithMessage: () -> Unit,
     onQuickActions: () -> Unit,
+    onShowSettingsDialog: () -> Unit,
 ) {
     Row(
         modifier = modifier,
@@ -45,28 +44,29 @@ fun Menu(
         MenuButton(
             modifier = Modifier
                 .padding(start = 16.dp),
-            title = "Workspace",
-            icon = painterResource("computer.svg"),
-            onClick = onGoToWorkspace,
-            fixedWidth = false,
+            title = "Open",
+            icon = painterResource("open.svg"),
+            onClick = onOpenAnotherRepository,
         )
 
         Spacer(modifier = Modifier.weight(1f))
 
+        MenuButton(
+            modifier = Modifier.padding(end = 4.dp),
+            title = "Fetch",
+            icon = painterResource("download.svg"),
+            onClick = { menuViewModel.fetchAll() },
+        )
+
         ExtendedMenuButton(
-            modifier = Modifier.padding(end = 8.dp),
+            modifier = Modifier.padding(end = 4.dp),
             title = "Pull",
             icon = painterResource("download.svg"),
             onClick = { menuViewModel.pull() },
-            extendedListItems = pullContextMenuItems(
-                onPullRebase = {
+            extendedListItems = pullContextMenuItems {
                 menuViewModel.pull(true)
-                },
-                onFetchAll = {
-                    menuViewModel.fetchAll()
             }
         )
-        )
 
         ExtendedMenuButton(
             title = "Push",
@@ -82,20 +82,27 @@ fun Menu(
             )
         )
 
-        Spacer(modifier = Modifier.width(24.dp))
+        Spacer(modifier = Modifier.width(32.dp))
 
         MenuButton(
+            modifier = Modifier.padding(end = 4.dp),
             title = "Branch",
             icon = painterResource("branch.svg"),
-            onClick = {
+        ) {
             onCreateBranch()
-            },
-        )
+        }
 
-        Spacer(modifier = Modifier.width(24.dp))
+        MenuButton(
+            title = "Merge",
+            icon = painterResource("merge.svg"),
+        ) {
+            onCreateBranch()
+        }
+
+        Spacer(modifier = Modifier.width(32.dp))
 
         ExtendedMenuButton(
-            modifier = Modifier.padding(end = 8.dp),
+            modifier = Modifier.padding(end = 4.dp),
             title = "Stash",
             icon = painterResource("stash.svg"),
             onClick = { menuViewModel.stash() },
@@ -107,18 +114,30 @@ fun Menu(
         MenuButton(
             title = "Pop",
             icon = painterResource("apply_stash.svg"),
-            onClick = { menuViewModel.popStash() },
-        )
+        ) { menuViewModel.popStash() }
 
         Spacer(modifier = Modifier.weight(1f))
 
+//        MenuButton(
+//            modifier = Modifier.padding(end = 4.dp),
+//            title = "Terminal",
+//            icon = painterResource("terminal.svg"),
+//            onClick = onQuickActions,
+//        )
+
         MenuButton(
-            title = "Quick actions",
-            modifier = Modifier.padding(end = 16.dp),
+            modifier = Modifier.padding(end = 4.dp),
+            title = "Actions",
             icon = painterResource("bolt.svg"),
-            fixedWidth = false,
             onClick = onQuickActions,
         )
+
+        MenuButton(
+            modifier = Modifier.padding(end = 16.dp),
+            title = "Settings",
+            icon = painterResource("settings.svg"),
+            onClick = onShowSettingsDialog,
+        )
     }
 }
 
@@ -128,36 +147,28 @@ fun MenuButton(
     enabled: Boolean = true,
     title: String,
     icon: Painter,
-    fixedWidth: Boolean = true,
     onClick: () -> Unit
 ) {
-    Row(
+    Column(
         modifier = modifier
             .ignoreKeyEvents()
             .clip(RoundedCornerShape(4.dp))
-            .background(MaterialTheme.colors.primary)
+            .background(MaterialTheme.colors.surface)
             .handMouseClickable { if (enabled) onClick() }
-            .run {
-                return@run if (fixedWidth) {
-                    this.width(92.dp)
-                } else
-                    this.padding(horizontal = 8.dp)
-            },
-        horizontalArrangement = Arrangement.Center,
-        verticalAlignment = Alignment.CenterVertically,
+            .size(56.dp),
+        horizontalAlignment = Alignment.CenterHorizontally,
+        verticalArrangement = Arrangement.Center,
     ) {
         Icon(
             painter = icon,
             contentDescription = title,
             modifier = Modifier
-                .padding(vertical = 2.dp)
-                .size(20.dp),
+                .size(24.dp),
             tint = MaterialTheme.colors.onPrimary,
         )
         Text(
             text = title,
-            style = MaterialTheme.typography.body2,
-            modifier = Modifier.padding(start = 2.dp, top = 8.dp, bottom = 8.dp),
+            style = MaterialTheme.typography.caption,
             maxLines = 1,
             textAlign = TextAlign.Center,
             color = MaterialTheme.colors.onPrimary,
@@ -178,29 +189,29 @@ fun ExtendedMenuButton(
 
     Row(
         modifier = modifier
-            .height(IntrinsicSize.Min)
+            .size(width = 64.dp, height = 56.dp)
             .ignoreKeyEvents()
             .clip(RoundedCornerShape(4.dp))
-            .background(MaterialTheme.colors.primary)
+            .background(MaterialTheme.colors.surface)
     ) {
-        Row(
+        Column(
             modifier = Modifier
-                .width(84.dp)
+                .fillMaxHeight()
+                .weight(1f)
                 .handMouseClickable { if (enabled) onClick() },
-            horizontalArrangement = Arrangement.Center,
-            verticalAlignment = Alignment.CenterVertically,
+            verticalArrangement = Arrangement.Center,
+            horizontalAlignment = Alignment.CenterHorizontally,
         ) {
             Icon(
                 painter = icon,
                 contentDescription = title,
                 modifier = Modifier
-                    .size(20.dp),
+                    .size(24.dp),
                 tint = MaterialTheme.colors.onPrimary,
             )
             Text(
                 text = title,
-                modifier = Modifier.padding(start = 2.dp, top = 8.dp, bottom = 8.dp),
-                style = MaterialTheme.typography.body2,
+                style = MaterialTheme.typography.caption,
                 color = MaterialTheme.colors.onPrimary,
                 maxLines = 1,
             )
@@ -208,13 +219,6 @@ fun ExtendedMenuButton(
 
         Box(
             modifier = Modifier
-                .fillMaxHeight()
-                .background(MaterialTheme.colors.onPrimary.copy(alpha = 0.3f))
-                .width(2.dp)
-        )
-        Box(
-            modifier = Modifier
-                .width(24.dp)
                 .fillMaxHeight()
                 .ignoreKeyEvents()
                 .handMouseClickable {
@@ -224,9 +228,10 @@ fun ExtendedMenuButton(
         ) {
 
             Icon(
-                Icons.Default.ArrowDropDown,
+                painterResource("expand_more.svg"),
                 contentDescription = null,
                 tint = MaterialTheme.colors.onPrimary,
+                modifier = Modifier.size(16.dp)
             )
 
             DropdownMenu(

src/main/kotlin/com/jetpackduba/gitnuro/ui/RepositoryOpen.kt

@@ -2,11 +2,9 @@
 
 package com.jetpackduba.gitnuro.ui
 
-import androidx.compose.foundation.ScrollState
 import androidx.compose.foundation.background
 import androidx.compose.foundation.focusable
 import androidx.compose.foundation.layout.*
-import androidx.compose.foundation.rememberScrollState
 import androidx.compose.material.Icon
 import androidx.compose.material.MaterialTheme
 import androidx.compose.material.Text
@@ -19,7 +17,6 @@ import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.input.key.onKeyEvent
 import androidx.compose.ui.input.pointer.PointerIcon
 import androidx.compose.ui.input.pointer.pointerHoverIcon
-import androidx.compose.ui.layout.onSizeChanged
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.text.style.TextOverflow
@@ -28,7 +25,6 @@ import com.jetpackduba.gitnuro.extensions.handMouseClickable
 import com.jetpackduba.gitnuro.git.DiffEntryType
 import com.jetpackduba.gitnuro.keybindings.KeybindingOption
 import com.jetpackduba.gitnuro.keybindings.matchesBinding
-import com.jetpackduba.gitnuro.theme.onBackgroundSecondary
 import com.jetpackduba.gitnuro.ui.components.ScrollableColumn
 import com.jetpackduba.gitnuro.ui.dialogs.*
 import com.jetpackduba.gitnuro.ui.diff.Diff
@@ -98,6 +94,7 @@ fun RepositoryOpenPage(
                 when (it) {
                     QuickActionType.OPEN_DIR_IN_FILE_MANAGER -> tabViewModel.openFolderInFileExplorer()
                     QuickActionType.CLONE -> onShowCloneDialog()
+                    QuickActionType.REFRESH -> tabViewModel.refreshAll()
                 }
             },
         )
@@ -132,13 +129,14 @@ fun RepositoryOpenPage(
                         Menu(
                             modifier = Modifier
                                 .padding(
-                                    vertical = 12.dp
+                                    vertical = 4.dp
                                 )
                                 .fillMaxWidth(),
                             onCreateBranch = { showNewBranchDialog = true },
                             onStashWithMessage = { showStashWithMessageDialog = true },
-                            onGoToWorkspace = { tabViewModel.selectUncommitedChanges() },
-                            onQuickActions = { showQuickActionsDialog = true }
+                            onOpenAnotherRepository = { openRepositoryDialog(tabViewModel) },
+                            onQuickActions = { showQuickActionsDialog = true },
+                            onShowSettingsDialog = onShowSettingsDialog
                         )
 
                         RepoContent(
@@ -148,7 +146,6 @@ fun RepositoryOpenPage(
                             repositoryState = repositoryState,
                             blameState = blameState,
                             showHistory = showHistory,
-                            onShowSettingsDialog = onShowSettingsDialog
                         )
                     }
                 }
@@ -201,7 +198,6 @@ fun RepoContent(
     repositoryState: RepositoryState,
     blameState: BlameState,
     showHistory: Boolean,
-    onShowSettingsDialog: () -> Unit,
 ) {
     if (showHistory) {
         val historyViewModel = tabViewModel.historyViewModel
@@ -221,7 +217,6 @@ fun RepoContent(
             selectedItem,
             repositoryState,
             blameState,
-            onShowSettingsDialog,
         )
     }
 }
@@ -267,24 +262,13 @@ fun MainContentView(
     selectedItem: SelectedItem,
     repositoryState: RepositoryState,
     blameState: BlameState,
-    onShowSettingsDialog: () -> Unit,
-
     ) {
     HorizontalSplitPane(
         splitPaneState = rememberSplitPaneState(initialPositionPercentage = 0.20f)
     ) {
         first(minSize = 180.dp) {
             Column {
-                val state: ScrollState = rememberScrollState()
-
-                val canBeScrolled by remember {
-                    derivedStateOf {
-                        state.maxValue > 0
-                    }
-                }
-
                 ScrollableColumn(
-                    state = state,
                     modifier = Modifier
                         .weight(1f),
                 ) {
@@ -294,20 +278,6 @@ fun MainContentView(
                     Stashes()
 //                TODO: Enable on 1.2.0 when fully implemented Submodules()
                 }
-
-                Column {
-                    if (canBeScrolled) {
-                        Box(
-                            Modifier
-                                .fillMaxWidth()
-                                .height(2.dp)
-                                .background(MaterialTheme.colors.onBackgroundSecondary.copy(alpha = 0.2f))
-                        )
-                    }
-                    SidePanelOption("Open another repository", "open.svg") { openRepositoryDialog(tabViewModel = tabViewModel) }
-                    SidePanelOption("Refresh", "refresh.svg") { tabViewModel.refreshAll() }
-                    SidePanelOption("Settings", "settings.svg", onShowSettingsDialog)
-                }
             }
         }
 

src/main/kotlin/com/jetpackduba/gitnuro/ui/context_menu/PullContextMenu.kt

@@ -2,19 +2,13 @@ package com.jetpackduba.gitnuro.ui.context_menu
 
 import androidx.compose.foundation.ExperimentalFoundationApi
 
-@OptIn(ExperimentalFoundationApi::class)
 fun pullContextMenuItems(
     onPullRebase: () -> Unit,
-    onFetchAll: () -> Unit,
 ): List<DropDownContentData> {
     return mutableListOf(
         DropDownContentData(
             label = "Pull with rebase",
             onClick = onPullRebase,
         ),
-        DropDownContentData(
-            label = "Fetch all",
-            onClick = onFetchAll,
-        ),
     )
 }

src/main/kotlin/com/jetpackduba/gitnuro/ui/dialogs/QuickActionsDialog.kt

@@ -35,6 +35,7 @@ fun QuickActionsDialog(
         listOf(
             QuickAction("code.svg", "Open repository in file manager", QuickActionType.OPEN_DIR_IN_FILE_MANAGER),
             QuickAction("download.svg", "Clone new repository", QuickActionType.CLONE),
+            QuickAction("refresh.svg", "Refresh repository data", QuickActionType.REFRESH),
         )
     }
 
@@ -124,5 +125,6 @@ data class QuickAction(val icon: String, val title: String, val type: QuickActio
 
 enum class QuickActionType {
     OPEN_DIR_IN_FILE_MANAGER,
-    CLONE;
+    CLONE,
+    REFRESH;
 }

src/main/resources/merge.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" enable-background="new 0 0 24 24" height="24px" viewBox="0 0 24 24" width="24px" fill="#000000"><g><rect fill="none" height="24" width="24"/></g><g><path d="M6.41,21L5,19.59l4.83-4.83c0.75-0.75,1.17-1.77,1.17-2.83v-5.1L9.41,8.41L8,7l4-4l4,4l-1.41,1.41L13,6.83v5.1 c0,1.06,0.42,2.08,1.17,2.83L19,19.59L17.59,21L12,15.41L6.41,21z"/></g></svg>

src/main/resources/terminal.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" enable-background="new 0 0 24 24" height="24px" viewBox="0 0 24 24" width="24px" fill="#000000"><g><rect fill="none" height="24" width="24"/></g><g><path d="M20,4H4C2.89,4,2,4.9,2,6v12c0,1.1,0.89,2,2,2h16c1.1,0,2-0.9,2-2V6C22,4.9,21.11,4,20,4z M20,18H4V8h16V18z M18,17h-6v-2 h6V17z M7.5,17l-1.41-1.41L8.67,13l-2.59-2.59L7.5,9l4,4L7.5,17z"/></g></svg>