From 806983db93c8309fa8cd9d566877af5abdfc6021 Mon Sep 17 00:00:00 2001 From: miruka Date: Fri, 9 Apr 2021 02:23:34 -0400 Subject: [PATCH] Use Instantiator for RoomList AtIndex keybinds Instantiator is better suited to the task: Repeater is specially made to load visual components, and that forced us to wrap the HShortcuts inside empty Items. --- src/gui/MainPane/RoomList.qml | 22 ++++++++-------------- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/src/gui/MainPane/RoomList.qml b/src/gui/MainPane/RoomList.qml index 27b8010b..b78f7492 100644 --- a/src/gui/MainPane/RoomList.qml +++ b/src/gui/MainPane/RoomList.qml @@ -269,25 +269,19 @@ HListView { onActivated: cycleUnreadRooms(true, true) && showItemLimiter.restart() } - Repeater { + Instantiator { model: Object.keys(window.settings.Keys.Accounts.AtIndex) - - Item { - HShortcut { - sequences: window.settings.Keys.Accounts.AtIndex[modelData] - onActivated: goToAccountNumber(parseInt(modelData, 10) - 1) - } + delegate: HShortcut { + sequences: window.settings.Keys.Accounts.AtIndex[modelData] + onActivated: goToAccountNumber(parseInt(modelData, 10) - 1) } } - Repeater { + Instantiator { model: Object.keys(window.settings.Keys.Rooms.AtIndex) - - Item { - HShortcut { - sequences: window.settings.Keys.Rooms.AtIndex[modelData] - onActivated: showAccountRoomAtIndex(parseInt(modelData,10) - 1) - } + delegate: HShortcut { + sequences: window.settings.Keys.Rooms.AtIndex[modelData] + onActivated: showAccountRoomAtIndex(parseInt(modelData, 10) - 1) } }