[ui] StatusMessages: corrections + reverse items order

Author: Alxiice

meshroom/ui/app.py

@@ -368,7 +368,7 @@ def forceUIUpdate(self):
 
     def showMessage(self, message, status=None, duration=5000):
         self._messageController.sendMessage(message, status, duration)
-    
+
     def _retrieveThumbnailPath(self, filepath: str) -> str:
         """
         Given the path of a project file, load this file and try to retrieve the path to its thumbnail, i.e. its

meshroom/ui/components/messaging.py

@@ -9,7 +9,7 @@ def __init__(self, msg, status=None):
         self.msg = msg
         self.status = status or "info"
         self.date = datetime.now()
-    
+
     def dateStr(self, fullDate=False):
         dateFormat = "%H:%M:%S"
         if fullDate:
@@ -21,24 +21,24 @@ class MessageController(QObject):
     """
     Handles messages sent from the Python side to the StatusBar component
     """
-    
+
     message = Signal(str, str, int)
     messagesChanged = Signal()  # Signal to notify when messages list changes
-    
+
     def __init__(self, parent):
         super().__init__(parent)
         self._messages = []
-    
+
     def sendMessage(self, msg, status, duration):
         """ Sends a message that will be displayed on the status bar """
         self.message.emit(msg, status, duration)
-    
+
     @Slot(str, str)
     def storeMessage(self, msg, status):
         """ Adds a new message in the stack """
         self._messages.append(Message(msg, status or "info"))
         self.messagesChanged.emit()  # Notify QML that messages have changed
-    
+
     def _getMessagesDict(self, fullDate=False):
         """ Get a dict with all stored messages """
         messages = []
@@ -49,11 +49,13 @@ def _getMessagesDict(self, fullDate=False):
                 "text": msg.msg,
             })
         return messages
-    
+
     def getMessages(self):
-        """ Get the messages with simple date infos """
-        return self._getMessagesDict()
-    
+        """ Get the messages with simple date infos.
+        Reverse the list to make sure we see the most recent item on top
+        """
+        return self._getMessagesDict()[::-1]
+
     @Slot(result=str)
     def getMessagesAsString(self):
         """ Return messages for clipboard copy 
@@ -64,12 +66,12 @@ def getMessagesAsString(self):
         for msg in self._messages:
             messages.append(f"{msg.dateStr(True)} [{msg.status.upper():<7}] {msg.msg}")
         return "\n".join(messages)
-    
+
     @Slot()
     def clearMessages(self):
         """ Clear all stored messages """
         self._messages.clear()
         self.messagesChanged.emit()
-    
+
     # Property to expose messages to QML
     messages = Property("QVariantList", getMessages, notify=messagesChanged)

meshroom/ui/qml/Controls/StatusBar.qml

@@ -103,7 +103,7 @@ RowLayout {
         // Add message to the message list
         _messageController.storeMessage(msg, status)
     }
-    
+
     Connections {
         target: _messageController
         function onMessage(message, color, duration) {

meshroom/ui/qml/Controls/StatusMessages.qml

@@ -25,7 +25,6 @@ ApplicationWindow {
     }
 
     function getBackgroundColor(status) {
-        // var color = Qt.darker(getColor(status), 1.2)
         var color = getColor(status)
         var alphaValue = status == "info" ? 0.05 : 0.1
         return Qt.rgba(color.r, color.g, color.b, alphaValue)
@@ -77,7 +76,7 @@ ApplicationWindow {
             }
 
             MaterialToolButton {
-                ToolTip.text: "Copy the content as a dict"
+                ToolTip.text: "Copy the messages"
                 text: MaterialIcons.content_copy
                 font.pointSize: 16
                 palette.base: systemPalette.base
@@ -107,6 +106,7 @@ ApplicationWindow {
             ListView {
                 id: messageListView
                 model: _messageController.messages
+                verticalLayoutDirection: ListView.TopToBottom
                 spacing: 5
 
                 delegate: Rectangle {
@@ -157,7 +157,7 @@ ApplicationWindow {
                 // Empty state
                 Text {
                     anchors.centerIn: parent
-                    text: "No messages to display"
+                    text: "No message to display"
                     color: Qt.darker(systemPalette.windowText, 1.5)
                     font.pointSize: 12
                     visible: messageListView.count === 0