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

Confusing (room) sections in the Explore Rooms dialog #12559

Closed
aleb opened this issue Feb 28, 2020 · 3 comments
Closed

Confusing (room) sections in the Explore Rooms dialog #12559

aleb opened this issue Feb 28, 2020 · 3 comments
Labels

Comments

@aleb
Copy link

aleb commented Feb 28, 2020

Description

Users should not have to bother about the room networks in the "Explore rooms" dialog. The UI should also explain what they are exactly.

Steps to reproduce

  • Click Explore button, notice the dialog

Screenshot 2020-02-28 at 10 50 08

There is a search field saying "Find a room..." and at the right there is a combobox with a list of some kind of room sections. This reminds of the various IRC networks. The search is limited to the rooms in the selected section.

It would be good to allow searching in all the sections at once, and make this the default. When listing the results, each room would list the corresponding section(s) and it would be possible this way to give details about what they represent exactly, for example by linking them to some representative page.

Version information

riot-web version: 1.5.10

@turt2live
Copy link
Member

pinging @vector-im/design because they're in the area at the moment

@jryans
Copy link
Collaborator

jryans commented Feb 28, 2020

This is definitely on our minds at the moment. We'll be looking over this whole area as part of #5194.

@nadonomy
Copy link
Contributor

Closing as this area of the UI has been iterated on since.

@jryans jryans removed the Z-UI/UX label Mar 8, 2021
t3chguy pushed a commit that referenced this issue Oct 17, 2024
* Fire `ModalManagerEvent.Closed` when a dialog is closed

* Listen to modal events in the RA

* Fix first RA test
JohnSimonsen pushed a commit to verji/element-web-v2 that referenced this issue Dec 20, 2024
* Upgrade dependency to [email protected]

* v3.100.0-rc.0

* Upgrade dependency to [email protected]

* Remove code smell assertion identified by Sonar (element-hq#12547)

* This assertion is unnecessary since the receiver accepts the original type of the expression

* Implement `client.getDomain()` null check

* Update comment since `AutoDiscovery.findClientConfig` may still throw

* Report verification and recovery state to posthog (element-hq#12516)

* Report verification and recovery state to posthog

* Fix CryptoApi import

* Fix js-sdk import

* Review: Use DeviceVerificationStatus instead of CrossSigningStatus

* Review: Clean condition to check secrets in 4S

* review: Fix redundent !!

* Deprecate `Tooltip.tsx`

* Use tooltip compound in `MessageComposer.tsx`

* Use tooltip compound in `ReadReceiptGroup` component

* Use tooltip compound in `ReadReceiptPerson` component

* Delete `useTooltip.tsx`

* Use compound tooltip in `ReactionRowButton`

* Upgrade dependency to [email protected]

* Migrate `TooltipTarget` usage to compound `Tooltip` (element-hq#12542)

* Use new tooltip in `RoomTopic.tsx`

* Use new tooltip in `MLocationBody.tsx`

* Fix room topic

* Update location snapshot

* Use new tooltip in `AppPermission.tsx`

* Remove `TooltipTarget`

* Add tests for `RoomTopic`

* Add tests for `ReadReceiptGroup.tsx`

* Add more tests for `ReadReceiptGroup.tsx`

* Add more test for `ReactionsRowButton.tsx`

* Add seedrandom dep to `resolutions` (element-hq#12552)

* Move seedrandom dep to `resolutions`

... to shut renovate up

* Reinstate `seedrandom` as dev dependency

If we just declare it in `resolutions`, it doesn't get installed at all,
because it is only a devDependency of `bloom-filters`.

* [create-pull-request] automated change (element-hq#12554)

Co-authored-by: github-merge-queue <[email protected]>

* Avoid using deprecated exports, fields, and duplicate code (element-hq#12555)

* Fix tabbedview breakpoint width (element-hq#12556)

Which should be 1024 according to the designs, not 768

* Tooltip: close field tooltip when ESC is pressed (element-hq#12553)

* Close field tooltip when ESC is pressed

* Use `Key.ESCAPE`

* Close the release announcement when a dialog is opened (element-hq#12559)

* Fire `ModalManagerEvent.Closed` when a dialog is closed

* Listen to modal events in the RA

* Fix first RA test

* Upgrade dependency to [email protected]

* v3.100.0-rc.1

* Upgrade dependency to [email protected]

* v3.100.0

---------

Co-authored-by: RiotRobot <[email protected]>
Co-authored-by: Ed Geraghty <[email protected]>
Co-authored-by: Valere <[email protected]>
Co-authored-by: Florian Duros <[email protected]>
Co-authored-by: Florian Duros <[email protected]>
Co-authored-by: David Baker <[email protected]>
Co-authored-by: Richard van der Hoff <[email protected]>
Co-authored-by: github-merge-queue <[email protected]>
Co-authored-by: Michael Telatynski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants