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

Fix quote shortcut for most keyboard layouts #9298

Merged
merged 3 commits into from
Sep 29, 2022

Conversation

owi92
Copy link
Contributor

@owi92 owi92 commented Sep 20, 2022

Addresses point 2 of element-hq/element-web#21067

This adds the shift key to the format quote tooltip and keyboard shortcut, which wasn't working on most if not all keyboard layouts because the greater than (>) key usually requires the user to also press shift.

before:

before

after:

after

Signed-off-by: Ole Wieners wieners@elan-ev.de

Notes: Add Shift key to FormatQuote keyboard shortcut

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

Here's what your changelog entry will look like:

✨ Features

  • Add Shift key to FormatQuote keyboard shortcut (#9298). Contributed by @owi92.

This adds the `shift` key to the format quote keyboard shortcut,
which wasn't working on most if not all keyboard layouts because
the `greater than` (>) key usually requires the user to press `shift`.
@owi92 owi92 requested a review from a team as a code owner September 20, 2022 16:11
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Sep 20, 2022
Copy link
Contributor

@SimonBrandner SimonBrandner left a comment

Choose a reason for hiding this comment

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

Thank you for working on this - LGTM codewise!

This is going to need a review from the design team - tagging them

@SimonBrandner SimonBrandner requested review from a team and removed request for justjanne, weeman1337 and duxovni September 20, 2022 16:18
@SimonBrandner SimonBrandner enabled auto-merge (squash) September 20, 2022 17:41
@SimonBrandner SimonBrandner added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Sep 20, 2022
@SimonBrandner SimonBrandner merged commit 3c5ff30 into matrix-org:develop Sep 29, 2022
@t3chguy t3chguy removed the request for review from SimonBrandner September 30, 2022 11:30
odelcroi added a commit to tchapgouv/tchap-web-v4 that referenced this pull request Oct 5, 2022
* Device manager - select all devices ([\#9330](matrix-org/matrix-react-sdk#9330)). Contributed by @kerryarchibald.
* New group call experience: Call tiles ([\#9332](matrix-org/matrix-react-sdk#9332)).
* Add Shift key to FormatQuote keyboard shortcut ([\#9298](matrix-org/matrix-react-sdk#9298)). Contributed by @owi92.
* Device manager - sign out of multiple sessions ([\#9325](matrix-org/matrix-react-sdk#9325)). Contributed by @kerryarchibald.
* Display push toggle for web sessions (MSC3890) ([\#9327](matrix-org/matrix-react-sdk#9327)).
* Add device notifications enabled switch ([\#9324](matrix-org/matrix-react-sdk#9324)).
* Implement push notification toggle in device detail ([\#9308](matrix-org/matrix-react-sdk#9308)).
* New group call experience: Starting and ending calls ([\#9318](matrix-org/matrix-react-sdk#9318)).
* New group call experience: Room header call buttons ([\#9311](matrix-org/matrix-react-sdk#9311)).
* Make device ID copyable in device list ([\#9297](matrix-org/matrix-react-sdk#9297)). Contributed by @duxovni.
* Use display name instead of user ID when rendering power events ([\#9295](matrix-org/matrix-react-sdk#9295)).
* Read receipts for threads ([\#9239](matrix-org/matrix-react-sdk#9239)). Fixes #23191.
* Fix device selection in pre-join screen for Element Call video rooms ([\#9321](matrix-org/matrix-react-sdk#9321)). Fixes #23331.
* Don't render a 1px high room topic if the room topic is empty ([\#9317](matrix-org/matrix-react-sdk#9317)). Contributed by @Arnei.
* Don't show feedback prompts when that UIFeature is disabled ([\#9305](matrix-org/matrix-react-sdk#9305)). Fixes #23327.
* Fix soft crash around unknown room pills ([\#9301](matrix-org/matrix-react-sdk#9301)). Fixes matrix-org/element-web-rageshakes#15465.
* Fix spaces feedback prompt wrongly showing when feedback is disabled ([\#9302](matrix-org/matrix-react-sdk#9302)). Fixes #23314.
* Fix tile soft crash in ReplyInThreadButton ([\#9300](matrix-org/matrix-react-sdk#9300)). Fixes matrix-org/element-web-rageshakes#15493.
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 Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants