[GH-ISSUE #3351] [FR] have a shortcut for text alignment #1486

Closed
opened 2026-03-23 20:49:57 +00:00 by mirror · 11 comments
Owner

Originally created by @annieappflowy on GitHub (Sep 6, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/3351

Originally assigned to: @Jayaprakash-dev on GitHub.

Description

have a shortcut for text alignment

Impact

Customers who frequently use text alignments in the editor

Additional Context

No response

Originally created by @annieappflowy on GitHub (Sep 6, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/3351 Originally assigned to: @Jayaprakash-dev on GitHub. ### Description have a shortcut for text alignment ### Impact Customers who frequently use text alignments in the editor ### Additional Context _No response_
Author
Owner

@MayurSMahajan commented on GitHub (Sep 6, 2023):

Ideally the following are shortcuts for text alignment:
Ctrl+L : align left
Ctrl+R : align right
Ctrl+E : align center

Ctrl+E is already used to format code.
Also on the web, Ctrl+R refreshes the page in the browser.

So I propose we use a new set of shortcuts for text alignment:
Ctrl+Alt+L : align left
Ctrl+Alt+R : align right
Ctrl+Alt+E : align center

This way all our text alignment commands will be uniform and won't interfere with any existing commands.
cc @annieappflowy

<!-- gh-comment-id:1708711723 --> @MayurSMahajan commented on GitHub (Sep 6, 2023): Ideally the following are shortcuts for text alignment: `Ctrl+L` : align left `Ctrl+R` : align right `Ctrl+E` : align center `Ctrl+E` is already used to format code. Also on the web, Ctrl+R refreshes the page in the browser. So I propose we use a new set of shortcuts for text alignment: `Ctrl+Alt+L` : align left `Ctrl+Alt+R` : align right `Ctrl+Alt+E` : align center This way all our text alignment commands will be uniform and won't interfere with any existing commands. cc @annieappflowy
Author
Owner

@sherazahmad720 commented on GitHub (Sep 10, 2023):

@annieappflowy can i work on this?

<!-- gh-comment-id:1712807649 --> @sherazahmad720 commented on GitHub (Sep 10, 2023): @annieappflowy can i work on this?
Author
Owner

@MayurSMahajan commented on GitHub (Sep 10, 2023):

Sure! Thanks @sherazahmad720 for helping out.

<!-- gh-comment-id:1712826534 --> @MayurSMahajan commented on GitHub (Sep 10, 2023): Sure! Thanks @sherazahmad720 for helping out.
Author
Owner

@Xazin commented on GitHub (Oct 25, 2023):

Hey @sherazahmad720, are you still working on this? Do you need help?

<!-- gh-comment-id:1779883565 --> @Xazin commented on GitHub (Oct 25, 2023): Hey @sherazahmad720, are you still working on this? Do you need help?
Author
Owner

@sherazahmad720 commented on GitHub (Oct 25, 2023):

@Xazin Sorry i stuck somewhere if you want you can continue on this

<!-- gh-comment-id:1780140712 --> @sherazahmad720 commented on GitHub (Oct 25, 2023): @Xazin Sorry i stuck somewhere if you want you can continue on this
Author
Owner

@annieappflowy commented on GitHub (Nov 5, 2023):

We could give you @sherazahmad720 more hints if you're keen to complete it.

<!-- gh-comment-id:1793748720 --> @annieappflowy commented on GitHub (Nov 5, 2023): We could give you @sherazahmad720 more hints if you're keen to complete it.
Author
Owner

@Xazin commented on GitHub (Nov 5, 2023):

@Xazin Sorry i stuck somewhere if you want you can continue on this

If you still want a go, feel free to mention me on our discord and I will guide you.

<!-- gh-comment-id:1793748973 --> @Xazin commented on GitHub (Nov 5, 2023): > @Xazin Sorry i stuck somewhere if you want you can continue on this If you still want a go, feel free to mention me on our discord and I will guide you.
Author
Owner

@Jayaprakash-dev commented on GitHub (Jan 5, 2024):

Hey @annieappflowy can I work on this?
I've already worked on it, and everything seems to be working smoothly.

One thing to note is that I haven't written a test for it yet. Would that be alright?

<!-- gh-comment-id:1879061811 --> @Jayaprakash-dev commented on GitHub (Jan 5, 2024): Hey @annieappflowy can I work on this? I've already worked on it, and everything seems to be working smoothly. One thing to note is that I haven't written a test for it yet. Would that be alright?
Author
Owner

@Xazin commented on GitHub (Jan 5, 2024):

Hey @annieappflowy can I work on this? I've already worked on it, and everything seems to be working smoothly.

One thing to note is that I haven't written a test for it yet. Would that be alright?

Hey @Jayaprakash-dev , I've assigned you.

Do you need help with adding tests for it? Feel free to mention me on our Discord (I have the same handle as on github) if you need assistance.

<!-- gh-comment-id:1879073694 --> @Xazin commented on GitHub (Jan 5, 2024): > Hey @annieappflowy can I work on this? I've already worked on it, and everything seems to be working smoothly. > > One thing to note is that I haven't written a test for it yet. Would that be alright? Hey @Jayaprakash-dev , I've assigned you. Do you need help with adding tests for it? Feel free to mention me on our Discord (I have the same handle as on github) if you need assistance.
Author
Owner

@Jayaprakash-dev commented on GitHub (Jan 5, 2024):

Hey @Xazin. Thanks for assigning it to me!

Sure I'll definitely reach out if I encounter any challenges. Could you provide a hint on where I should add the test in terms of the file or module? That would be super helpful.

<!-- gh-comment-id:1879089112 --> @Jayaprakash-dev commented on GitHub (Jan 5, 2024): Hey @Xazin. Thanks for assigning it to me! Sure I'll definitely reach out if I encounter any challenges. Could you provide a hint on where I should add the test in terms of the file or module? That would be super helpful.
Author
Owner

@Xazin commented on GitHub (Jan 5, 2024):

Hey @Xazin. Thanks for assigning it to me!

Sure I'll definitely reach out if I encounter any challenges. Could you provide a hint on where I should add the test in terms of the file or module? That would be super helpful.

Sure thing!

You can add a test for the alignment shortcut in this file: appflowy_flutter/integration_test/document/document_alignment_test.dart- It currently holds the corresponding test for the toolbar. You can see how the expect is used, it should be similar to the test you will have to write.

<!-- gh-comment-id:1879092087 --> @Xazin commented on GitHub (Jan 5, 2024): > Hey @Xazin. Thanks for assigning it to me! > > Sure I'll definitely reach out if I encounter any challenges. Could you provide a hint on where I should add the test in terms of the file or module? That would be super helpful. Sure thing! You can add a test for the alignment shortcut in this file: `appflowy_flutter/integration_test/document/document_alignment_test.dart`- It currently holds the corresponding test for the toolbar. You can see how the `expect` is used, it should be similar to the test you will have to write.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#1486
No description provided.