mirror of
https://github.com/AppFlowy-IO/AppFlowy.git
synced 2026-03-24 12:56:59 +00:00
[GH-ISSUE #7586] [Bug] "turn into" using shortcuts behave differently #3363
Labels
No labels
2024
2025
2026
acct mgmt
AI
automation
bug
calendar
ci
CJK
cloud
code-block
collaboration
copy-paste
database
data migration
data sync
deploy
desktop
develop
develop
documentation
duplicate
editor
editor-plugin
emoji
export
files
flutter-only
follow-up
formula
good first issue for devs
good first issue for experienced devs
grid
hacktoberfest
HACKTOBERFEST-ACCEPTED
help wanted
i18n
icons
images
importer
improvements
infra
install
integrations
IR
kanban board
login
look and joy
mentorship
mobile
mobile
needs design
new feature
new feature
non-coding
notes
notifications
onboarding
organization
P0+
permission
platform-linux
platform-mac
platform-windows
plugins
program
pull-request
Q1 25
Q1 26
Q2 24
Q2 25
Q3 24
Q3 25
Q4 24
Q4 25
react
regression
rust
rust
Rust-only
Rust-only
Rust-starter
Rust-starter
self-hosted
shortcuts
side panel
slash-menu
sync v2
table
tablet
task
tauri
templates
tests
themes
translation
v0.5.6
v0.5.8
v0.5.9
v0.6.0
v0.6.1
v0.6.4
v0.6.7
v0.6.8
v0.7.1
v0.7.4
v0.7.4
v0.7.5
v0.7.6
v0.7.7
v0.7.8
v0.8.0
v0.8.4
v0.8.5
v0.8.9
web
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
AppFlowy-IO/AppFlowy#3363
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @WagnerGFX on GitHub (Mar 22, 2025).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/7586
Originally assigned to: @asjqkkkk on GitHub.
Bug Description
Using the shortcuts for "Toggle Heading" makes any child text blocks disappear.
I'm not sure if these shortcuts were expected to toggle between Heading and text or to transform into a Toggle Heading. So I'm assuming it should behave the same as using "turn into Heading"
How to Reproduce
Option A: Use the left handler and turn it into H1, H2 or H3
Option B: Use the right click menu to turn it into H1, H2 or H3
Option C: Use the shortcut for Toggle Heading 1, 2 or 3
Using the shortcut will hide the second text block, while the others will just lower its indentation.
Expected Behavior
Using the left handle menu, the right click menu and shortcuts should all behave the exactly same.
The shortcut option is the offender for hiding the block's child content.
I'm just not sure if the fix is for the child text to stay visible, or if the shortcut was supposed to turn into a Toggle Heading.
In any case, I also suggest changing the shortcut names to "Toggle into ..." or something similar to prevent confusion.
Operating System
Windows 10
AppFlowy Version(s)
0.8.7
Screenshots
No response
Additional Context
No response
@WagnerGFX commented on GitHub (Mar 25, 2025):
The naming is secondary, the main issue I reported was about the shortcut making the child blocks disappear, while using "turn into" didn't.
That PR seems to only change the name. Was the behavior fixed in another PR?
@asjqkkkk commented on GitHub (Mar 26, 2025):
HI @WagnerGFX , do you mean this issue : https://github.com/AppFlowy-IO/AppFlowy/issues/7585 ?
it will be fixed by https://github.com/AppFlowy-IO/AppFlowy/pull/7598
@WagnerGFX commented on GitHub (Mar 26, 2025):
Not exactly. I did open a few issues with the "theme" of data loss. But they each happen in different situations.
The case for this specific issue is that using one of the "Toggle Heading" shortcuts makes any child blocks disappear.
But the "Toggle" in those shortcuts just means the block will be changed into that or back into normal text, so those are shortcuts to simple Headings.
And this issue only happens for the shortcut, since using the left handler or the context menu all behave normally.
But I have no idea how the code works internally 😅, so that fix could be what solves all those cases.
When the new version gets released, I'll test everything to see if there are any problems left.
Thank you for the feedback 👍
@WagnerGFX commented on GitHub (Apr 3, 2025):
I tested on v0.8.8 and the issue still remains. Here is a video to see how it happens:
https://github.com/user-attachments/assets/0e2e60fe-c282-47fa-8ff1-a98516b051c2
Also, related to renaming the shortcuts, it's also still the same as before.
Although renaming shortcuts is not the point of this issue, I thought it would be best to inform about it: