Skip to content

feat: Add standard markdown shortcuts#2843

Open
ION606 wants to merge 3 commits into
krille-chan:mainfrom
ION606:feat/shortcuts
Open

feat: Add standard markdown shortcuts#2843
ION606 wants to merge 3 commits into
krille-chan:mainfrom
ION606:feat/shortcuts

Conversation

@ION606

@ION606 ION606 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Added standard markdown shortcuts to the chat (ctrl+i, ctrl+a, ctrl+b, etc)

Thank you so much for your contribution to FluffyChat ❤️❤️❤️

Pull Request has been tested on:

  • Android
  • iOS
  • Browser (Chromium based)
  • Browser (Firefox based)
  • Browser (WebKit based)
  • Desktop Linux
  • Desktop Windows
  • Desktop macOS

@ION606 ION606 requested a review from krille-chan as a code owner April 9, 2026 17:53
Comment thread lib/pages/chat/chat.dart Outdated
if (evt is KeyDownEvent &&
(HardwareKeyboard.instance.isControlPressed ||
HardwareKeyboard.instance.isMetaPressed) &&
!PlatformInfos.isMobile) {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why not enable this on mobile too? This would also block iPads with a keyboard or Samsung Dex or the new Android 16 desktop mode or other scenarios

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

mostly because I didn't really think of it. Idrk anything about ipad/IOS qwq

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

after looking at it, I think this would be handled by HardwareKeyboard.instance.isControlPressed which should work with those

Comment thread lib/pages/chat/chat.dart
if (evt.logicalKey == LogicalKeyboardKey.keyY) {
undoController.redo();
return KeyEventResult.handled;
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks very handy. 🤔 Actually I would like to prefer having an integration test for this as well to make sure this never breaks. Is this possible on Android?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it...should be...I think....I'll test it today 😅

@krille-chan

Copy link
Copy Markdown
Owner

@ION606 are you still going to work on this? CI is failing

@ION606

ION606 commented May 3, 2026

Copy link
Copy Markdown
Contributor Author

@ION606 are you still going to work on this? CI is failing

oh yea I forgot. Sure I can work on it tmw, ditto with the rest of them

@ION606 ION606 force-pushed the feat/shortcuts branch from 30f308c to 2ec9ccf Compare May 4, 2026 00:27
@krille-chan

Copy link
Copy Markdown
Owner

@ION606 formatting fails

@ION606

ION606 commented May 20, 2026

Copy link
Copy Markdown
Contributor Author

@ION606 formatting fails

I'll try to get to it after work today qwq

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants