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

[Labs] Refresh UI for keybindings and prep for customisable keybindings #5758

Closed

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Mar 13, 2021

This PR adds a new UI for keybindings hidden under labs. It also adds keybinding setting entries. Those will be useful for customizable keybindings in later PRs.

Well, this is my biggest PR yet. Let's see how badly it goes 😅

Note: The design review can be skipped here since this is behind labs.

Screenshot_20210313_132942

Element-web PR

Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
TravisR will fix this nightmare

Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner changed the title Refresh UI for keybindings and prep for customisable keybindings [LABS] Refresh UI for keybindings and prep for customisable keybindings Mar 17, 2021
@SimonBrandner SimonBrandner marked this pull request as ready for review March 26, 2021 18:56
@jryans jryans requested a review from a team March 27, 2021 15:03
@jryans jryans changed the title [LABS] Refresh UI for keybindings and prep for customisable keybindings [Labs] Refresh UI for keybindings and prep for customisable keybindings Mar 27, 2021
Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

Looks like a great PR to me! Thank you for the contribution
Going to wait for Travis' comment and add that back to the review queue as it's probably best if one other member of the team lay their eyes on this before we merge it

src/i18n/strings/en_EN.json Show resolved Hide resolved
src/components/views/elements/KeyboardShortcut.tsx Outdated Show resolved Hide resolved
src/settings/Settings.ts Outdated Show resolved Hide resolved
@germain-gg germain-gg requested a review from a team April 9, 2021 14:53
SimonBrandner and others added 4 commits April 9, 2021 16:57
Co-authored-by: Germain <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner requested review from germain-gg and removed request for a team April 9, 2021 16:38
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@turt2live turt2live requested review from turt2live and t3chguy and removed request for turt2live May 24, 2021 15:11
@SimonBrandner SimonBrandner added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Aug 4, 2021
@SimonBrandner SimonBrandner requested review from dbkr and removed request for dbkr August 5, 2021 11:21
@germain-gg germain-gg removed their request for review October 21, 2021 07:39
@SimonBrandner
Copy link
Contributor Author

I am going to close this as the PR is big and outdated by now. That makes it hard to continue working on it and review it. I instead choose a less radical approach of slowly moving toward customizable keybindings, first PR that does that is #7198

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants