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

Switch room explorer list to CSS grid #5551

Merged

Conversation

weeman1337
Copy link
Contributor

Solves the issue that languages require different spacing for the buttons.
Keyboard navigation works like before.

before

before

after

English

after_en

Français

after_fr

Closes element-hq/element-web#15347

@weeman1337 weeman1337 force-pushed the fix-15347-explore-rooms-spacing branch from 3e7b296 to 73c3a5e Compare January 19, 2021 17:50
@jryans jryans requested a review from a team January 21, 2021 13:35
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

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

Thanks for working on this, looks like a great improvement! 😄

Just one point about code style. Please click re-request wheel on the top right after you update the PR to ensure I see it promptly.

src/components/structures/RoomDirectory.js Outdated Show resolved Hide resolved
@weeman1337 weeman1337 force-pushed the fix-15347-explore-rooms-spacing branch 2 times, most recently from 586e1e6 to 2f3570c Compare January 25, 2021 17:48
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

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

Thanks for working this, looks good to me. 😄

@jryans
Copy link
Collaborator

jryans commented Jan 27, 2021

Ah curious, the end to end tests failed... I have retried them, let's see what happens.

@weeman1337
Copy link
Contributor Author

@jryans is there anything I can do? Looks like some timeouts are happening.

@t3chguy
Copy link
Member

t3chguy commented Feb 1, 2021

Try pulling in latest develop e.g git pull upstream/develop then git push

@jryans
Copy link
Collaborator

jryans commented Feb 1, 2021

@weeman1337 Yes, looks like the tests should pass once you merge in latest changes from develop.

Signed-off-by: Michael Weimann <mail@michael-weimann.eu>
@weeman1337 weeman1337 force-pushed the fix-15347-explore-rooms-spacing branch from 2f3570c to f0f6e46 Compare February 6, 2021 13:45
@weeman1337
Copy link
Contributor Author

@jryans done. Today I regret to not having finished this before the fosdem. The room list - urgh

@jryans jryans merged commit 5abced7 into matrix-org:develop Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Rejoindre" ("Join" in french) button in explore room modal doesn't fit
3 participants