Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Small delight tweaks to improve rough corners in the app #5418

Merged
merged 10 commits into from
Nov 18, 2020

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Nov 11, 2020

Fixes element-hq/element-web#9251
Fixes element-hq/element-web#14870
Fixes element-hq/element-web#12641

  • Fix left panel layout being wrong when filtering with 0 rooms

  • Fix room list join rooms prompt not showing up due to missing updates

  • Pass filter text when clicking explore/dm prompt

  • Iterate the filtering prompt
    image

  • Update Room List filter copy
    image

  • Tweak registration based on user feedback
    image
    image

@t3chguy t3chguy changed the title T3chguy/fix/muggle hustle Small delight tweaks to improve rough corners in the app Nov 11, 2020
@t3chguy t3chguy requested review from a team November 11, 2020 17:37
@t3chguy t3chguy marked this pull request as ready for review November 11, 2020 17:37
Copy link
Contributor

@niquewoodhouse niquewoodhouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all looks really good, thanks, we'll iterate after more testing/as part of other issues

Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise lgtm

// the email anyways, and so we don't cause things to jump around. In
// theory, the user would see the user pop up and think "ah yes, that
// person!"
const profile = await MatrixClientPeg.get().getProfileInfo(lookup.mxid);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may want to discard results that arrive out of order - as-you-type searching can be a bit subtle to get right: there should be code to crib from elsewhere (we should probably make it a utility class or something).

Copy link
Member Author

@t3chguy t3chguy Nov 18, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yep, that should do it!

@t3chguy t3chguy requested a review from dbkr November 18, 2020 10:02
@t3chguy t3chguy merged commit 9b21437 into develop Nov 18, 2020
@t3chguy t3chguy deleted the t3chguy/fix/muggle-hustle branch November 18, 2020 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants