PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Fix room list filtering weird case sensitivity

Browse source 
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
This commit is contained in:
Michael Telatynski 2019-12-21 11:13:37 +00:00
parent 24d0d0d596
commit e936f7eb09
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/components/views/rooms/RoomList.js
View file

@ -589,15 +589,17 @@ module.exports = createReactClass({
_applySearchFilter: function(list, filter) { _applySearchFilter: function(list, filter) {
if (filter === "") return list; if (filter === "") return list;
const fuzzyFilter = utils.removeHiddenChars(filter).toLowerCase();
const lcFilter = filter.toLowerCase(); const lcFilter = filter.toLowerCase();
// apply toLowerCase before and after removeHiddenChars because different rules get applied
// e.g M -> M but m -> n, yet some unicode homoglyphs come out as uppsercase, e.g 𝚮 -> H
const fuzzyFilter = utils.removeHiddenChars(lcFilter).toLowerCase();
// case insensitive if room name includes filter, // case insensitive if room name includes filter,
// or if starts with `#` and one of room's aliases starts with filter // or if starts with `#` and one of room's aliases starts with filter
return list.filter((room) => { return list.filter((room) => {
if (filter[0] === "#" && room.getAliases().some((alias) => alias.toLowerCase().startsWith(lcFilter))) { if (filter[0] === "#" && room.getAliases().some((alias) => alias.toLowerCase().startsWith(lcFilter))) {
return true; return true;
} }
return room.name ? utils.removeHiddenChars(room.name).toLowerCase().includes(fuzzyFilter) : false; return room.name && utils.removeHiddenChars(room.name.toLowerCase()).toLowerCase().includes(fuzzyFilter);
}); });
}, },