Add "Filename first" path display style

Author: zachrammell

gitfourchette/filelists/filelist.py

@@ -72,46 +72,84 @@ def paint(self, painter: QPainter, option: QStyleOptionViewItem, index: QModelIn
         if font:
             painter.setFont(font)
         fullText = index.data(Qt.ItemDataRole.DisplayRole)
-        text = painter.fontMetrics().elidedText(fullText, option.textElideMode, textRect.width())
+        elideMode = Qt.TextElideMode.ElideRight if settings.prefs.pathDisplayStyle == PathDisplayStyle.FileNameFirst else option.textElideMode
+        text = painter.fontMetrics().elidedText(fullText, elideMode, textRect.width())
 
         # Split path into directory and filename for better readability
-        dirPortion = None
-        filePortion = None
+        firstPortion = None
+        secondPortion = None
+        firstColor = QPalette.ColorRole.PlaceholderText
+        secondColor = QPalette.ColorRole.WindowText
+
+        # Determine split based on style
+        isFileNameFirst = settings.prefs.pathDisplayStyle == PathDisplayStyle.FileNameFirst
+
+        if isFileNameFirst:            
+            # Rely on the raw path to identify the filename, as filenames can contain spaces
+            fullPath = index.data(FileListModel.Role.FilePath)
+            fName = os.path.basename(fullPath)
+            
+            prefix = fName + " "
+            if not text.startswith(prefix) and '\u2026' in text:
+                ellipsisPos = text.find('\u2026')
+                if fName.startswith(text[:ellipsisPos]):
+                    prefix = text[:ellipsisPos + 1]
 
-        if '/' in fullText:
-            slashesInFull = fullText.count('/')
-            slashesInElided = text.count('/')
+            if text.startswith(prefix):
+                 firstPortion = prefix
+                 secondPortion = text[len(prefix):]
+            else:
+                 firstPortion = text
+                 secondPortion = ""
+            
+            firstColor = QPalette.ColorRole.WindowText
+            secondColor = QPalette.ColorRole.PlaceholderText
 
-            if slashesInFull > slashesInElided:
-                # A slash was elided - gray everything up to the ellipsis
-                ellipsisPos = text.find('\u2026')
-                dirPortion = text[:ellipsisPos + 1]
-                filePortion = text[ellipsisPos + 1:]
-            elif slashesInElided > 0:
-                # No slash elided - gray up to the last slash
-                lastSlash = text.rfind('/')
-                dirPortion = text[:lastSlash + 1]
-                filePortion = text[lastSlash + 1:]
-
-        if dirPortion is not None:
-            textColor = QPalette.ColorRole.WindowText if not isSelected else QPalette.ColorRole.HighlightedText
-            dirColor = QPalette.ColorRole.PlaceholderText if not isSelected else textColor
-
-            # Draw directory with muted color
-            mutedColor = option.palette.color(colorGroup, dirColor)
-            if isSelected:
-                mutedColor.setAlphaF(.7)
-            painter.setPen(mutedColor)
-            painter.drawText(textRect, option.displayAlignment, dirPortion)
-
-            # Draw filename with normal color
-            painter.setPen(option.palette.color(colorGroup, textColor))
-            dirWidth = painter.fontMetrics().horizontalAdvance(dirPortion)
-            fileRect = QRect(textRect)
-            fileRect.setLeft(textRect.left() + dirWidth)
-            painter.drawText(fileRect, option.displayAlignment, filePortion)
         else:
-            painter.drawText(textRect, option.displayAlignment, text)
+            if '/' in fullText:
+                slashesInFull = fullText.count('/')
+                slashesInElided = text.count('/')
+
+                if slashesInFull > slashesInElided:
+                    # A slash was elided - gray everything up to the ellipsis
+                    ellipsisPos = text.find('\u2026')
+                    firstPortion = text[:ellipsisPos + 1]
+                    secondPortion = text[ellipsisPos + 1:]
+                elif slashesInElided > 0:
+                    # No slash elided - gray up to the last slash
+                    lastSlash = text.rfind('/')
+                    firstPortion = text[:lastSlash + 1]
+                    secondPortion = text[lastSlash + 1:]
+            
+            if firstPortion is None:
+                 firstPortion = ""
+                 secondPortion = text
+            
+            firstColor = QPalette.ColorRole.PlaceholderText
+            secondColor = QPalette.ColorRole.WindowText
+
+        # Draw the parts
+        if firstPortion:
+            fg1 = option.palette.color(colorGroup, firstColor if not isSelected else QPalette.ColorRole.HighlightedText)
+            if firstColor == QPalette.ColorRole.PlaceholderText and isSelected:
+                 fg1 = QColor(option.palette.color(colorGroup, QPalette.ColorRole.HighlightedText))
+                 fg1.setAlphaF(0.7)
+            
+            painter.setPen(fg1)
+            painter.drawText(textRect, option.displayAlignment, firstPortion)
+            
+            # Prepare rect for second part
+            part1Width = painter.fontMetrics().horizontalAdvance(firstPortion)
+            textRect.setLeft(textRect.left() + part1Width)
+
+        if secondPortion:
+            fg2 = option.palette.color(colorGroup, secondColor if not isSelected else QPalette.ColorRole.HighlightedText)
+            if secondColor == QPalette.ColorRole.PlaceholderText and isSelected:
+                 fg2 = QColor(option.palette.color(colorGroup, QPalette.ColorRole.HighlightedText))
+                 fg2.setAlphaF(0.7)
+
+            painter.setPen(fg2)
+            painter.drawText(textRect, option.displayAlignment, secondPortion)
 
         # Highlight search term
         if searchTerm and searchTerm in fullText.lower():
@@ -122,8 +160,11 @@ def paint(self, painter: QPainter, option: QStyleOptionViewItem, index: QModelIn
             else:
                 needleLen = len(searchTerm)
 
+            # Use original textRect (before we advanced it for the second part)
+            textRect.setLeft(textRect.left() - part1Width if firstPortion else 0)
             SearchBar.highlightNeedle(painter, textRect, text, needlePos, needleLen)
 
+
         painter.restore()
 
 

gitfourchette/toolbox/pathutils.py

@@ -14,6 +14,7 @@ class PathDisplayStyle(enum.IntEnum):
     FullPaths = 1
     AbbreviateDirs = 2
     FileNameOnly = 3
+    FileNameFirst = 4
 
 
 def compactPath(path: str) -> str:
@@ -33,6 +34,11 @@ def abbreviatePath(path: str, style: PathDisplayStyle = PathDisplayStyle.FullPat
             else:
                 splitLong[i] = splitLong[i][0]
         return '/'.join(splitLong)
+    elif style == PathDisplayStyle.FileNameFirst:
+        split = path.rsplit('/', 1)
+        if len(split) == 1:
+            return path
+        return split[-1] + ' ' + split[0]
     elif style == PathDisplayStyle.FileNameOnly:
         return path.rsplit('/', 1)[-1]
     else:

gitfourchette/trtables.py

@@ -195,6 +195,7 @@ def _init_enums():
                 PathDisplayStyle.FullPaths      : _("Full paths"),
                 PathDisplayStyle.AbbreviateDirs : _("Abbreviate directories"),
                 PathDisplayStyle.FileNameOnly   : _("Show filename only"),
+                PathDisplayStyle.FileNameFirst  : _("Filename first"),
             },
 
             AuthorDisplayStyle: {

test/test_filelist.py

@@ -498,6 +498,9 @@ def testFileListChangePathDisplayStyle(tempDir, mainWindow):
     triggerContextMenuAction(rw.committedFiles.viewport(), "path display style/full")
     assert ["c/c2-2.txt"] == qlvGetRowData(rw.committedFiles)
 
+    triggerContextMenuAction(rw.committedFiles.viewport(), "path display style/name first")
+    assert ["c2-2.txt c"] == qlvGetRowData(rw.committedFiles)
+
 
 def testFileListShowInFolder(tempDir, mainWindow):
     wd = unpackRepo(tempDir)