matty/peertube-plugin-livechat
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Implement #305: UI/UX improvement:

Browse Source 
Don't group "followup" messages, and always display avatar/nickname/datetime besides messages, even when it is the same user as previous one.
This commit is contained in:
John Livingston 2024-01-31 18:22:26 +01:00
parent 5d4bfceb73
commit bfef57b1a5
No known key found for this signature in database
GPG Key ID: B17B5640CE66CDBC
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGELOG.md
View File

@ -10,9 +10,10 @@
* UX: add a label ('Choose a nickname to enter') for the anonymous nickname prompt. Fix #287.
* Translation updates: Arabic, German, French, Swedish, Spanish.
* New Swedish translations.
* UI/UX improvments:
* UI/UX improvements:
* hide nickname changes when previous nickname was like "Anonymous 12345". Helps to improve performances when massive anonymous users are joining (#138), and prevent displaying unnecessary messages (#111).
* display anonymous users at the end of the participants list (Fix #136)
* don't group "followup" messages, and always display avatar/nickname/datetime besides messages, even when it is the same user as previous one (#305)
* Using patched ConverseJS for performance improvment (related to #96):
* debounce MUC sidebar rendering in ConverseJS (Fix #138)
* force history pruning, even if messages keep coming (Fix #140)

5
conversejs/builtin.ts
View File

@ -129,6 +129,11 @@ window.initConverse = async function initConverse (initConverseParams: InitConve
return this.__super__.getActionInfoMessage(code, nick, actor)
}
},
ChatRoomMessage: {
/* By default, ConverseJS groups messages from the same users for a 10 minutes period.
* This make no sense in a livechat room. So we override isFollowup to ignore. */
isFollowup: function isFollowup () { return false }
},
ChatRoomOccupants: {
comparator: function (this: any, occupant1: any, occupant2: any): Number {
// Overriding Occupants comparators, to display anonymous users at the end of the list.