Bug 460601 - Use icons instead of names in accountList?
Summary: Use icons instead of names in accountList?
Status: RESOLVED FIXED
Alias: None
Product: NeoChat
Classification: Applications
Component: General (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Fella
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-10-17 15:07 UTC by Link Dupont
Modified: 2022-10-23 16:05 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Link Dupont 2022-10-17 15:07:09 UTC
The accountList RowLayout is using a Repeater to create Kirigami.BasicListItem entries in the RowLayout. This works fine with 2 accounts, but the more accounts you add, the more difficult it becomes to differentiate between items. For example, I have 4 accounts. Two of my accounts begin with the same account name, so they are indiscernible when the list truncates the text labels significantly. This could be solved either by using account icons in place of the localUserId, or by switching to a ColumnLayout if the number of accounts is > 3.
Comment 1 Link Dupont 2022-10-17 15:08:20 UTC
Or what about a combobox to switch accounts?
Comment 2 Nate Graham 2022-10-17 19:45:58 UTC
+1 for a combobox.
Comment 3 Tobias Fella 2022-10-17 19:49:30 UTC
We've made some plans at akademy about a new design that would involve something like an overlaysheet (conceptually, probably not in the implementation) as part of the account switching. i can't make any promises about when that will happen though
Comment 4 Link Dupont 2022-10-17 20:50:53 UTC
I swapped the RowLayout for a ColumnLayout. It was incredibly easy, and makes the view much more usable. I'll try switching it for a ComboBox and see how that works. Either way, it's an improvement over the current RowLayout until a new UI design can be implemented.
Comment 5 Tobias Fella 2022-10-23 16:05:56 UTC
Git commit ef255243ecc3c3e8839f800ad5d218a183905cd0 by Tobias Fella, on behalf of Link Dupont.
Committed on 23/10/2022 at 15:51.
Pushed by tfella into branch 'master'.

Use ComboBox as AccountRegistry view

Replace the RowLayout with a ComboBox. This permits a higher number
of accounts to be used in the application without reducing the
usability.

M  +10   -20   imports/NeoChat/Page/RoomListPage.qml
M  +10   -3    src/neochataccountregistry.cpp
M  +6    -0    src/neochataccountregistry.h

https://invent.kde.org/network/neochat/commit/ef255243ecc3c3e8839f800ad5d218a183905cd0