Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Search from room options if not available #4641

Closed
daniellekirkwood opened this issue Dec 6, 2021 · 2 comments
Closed

Remove Search from room options if not available #4641

daniellekirkwood opened this issue Dec 6, 2021 · 2 comments
Assignees
Labels
A-Search T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Ready This issue is ready for development Z-WTF WTF moment: High Impact, Low Effort

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Dec 6, 2021

This issue is: Ready to Build

Problem

When a user clicks the overflow menu, finds Search, and clicks it, they are told it is not available. This is frustrating

Solution

If an action (like search) is not available because of something we did (or don't do) then it should not be viewable by users. Remove the Search button if we do not provide search.

Note: If Search is unavailable due to the room type or configuration set by the user then Search should be available from the list (as this is the users limitation, not ours).

@daniellekirkwood daniellekirkwood added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-WTF WTF moment: High Impact, Low Effort labels Dec 6, 2021
@daniellekirkwood daniellekirkwood added the Z-Ready This issue is ready for development label Jan 26, 2022
@bmarty bmarty added this to the WTF ready list milestone Jan 31, 2022
@mnaturel mnaturel self-assigned this Feb 2, 2022
@mnaturel
Copy link
Contributor

mnaturel commented Feb 2, 2022

Hi @daniellekirkwood , I tried to reproduce this issue and investigate the corresponding source code. The only scenario I found in which the search is said to be not available is when the room is encrypted (see the capture below). I guess we are in the case of room configuration meaning the search should still be available in the list if I understood well.

dialog-issue-4641

Just to be sure: we only have this issue in a room screen, right ? If yes, should we still do something ?

@bmarty
Copy link
Member

bmarty commented Feb 2, 2022

@mnaturel yes, I think we just want to hide the search MenuItem for encrypted rooms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Search T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Ready This issue is ready for development Z-WTF WTF moment: High Impact, Low Effort
Projects
None yet
Development

No branches or pull requests

4 participants