🐛 Markdown Toolbar Formatting Issues (Bold / Italic / Strikethrough / Code) buttons doesn't get pressed even after clicking

[schourasia750]

🐛 Markdown Toolbar Buttons Do Not Toggle Properly

Description

The markdown formatting toolbar buttons (Bold, Italic, Strikethrough, Inline Code, Multiline Code) do not behave as toggle buttons.

Currently, when a formatting button is pressed, it inserts the corresponding markdown symbols but the button itself does not remain in an active state.

This makes the formatting experience inconsistent and confusing for users.

---

🔴 Issue: Button Activation Behavior

Formatting buttons should behave as toggle buttons.

Expected behavior:

• When pressed once → formatting mode becomes active
• When pressed again → formatting mode becomes inactive
• The button should visually indicate when it is active

Current behavior:

• Pressing the button only inserts markdown symbols
• The button does not stay in an active state
• Users cannot clearly see whether formatting is enabled or disabled

---

🧪 Affected Formatting Buttons

• Bold (**)
• Italic (*)
• Strikethrough (~~)
• Inline Code (`)
• Multiline Code (```)

---

💡 Suggested Improvement

• Implement proper toggle state for formatting buttons
• Provide visual feedback (active/pressed state)
• Ensure pressing the button again disables the formatting

---

Environment

• Latest develop branch
• Reproducible in the message composer

---

Note

Other formatting issues such as nested or duplicate markdown symbols have already been resolved, so this issue only focuses on the button toggle behavior.

[gunapriya07]

Hi @schourasia750 👋 I’d like to work on this issue. Please let me know if I can proceed. Thanks!

[schourasia750]

@gunapriya07 Yes you can proceed as I am not working on this right now

[gunapriya07]

@schourasia750 , Thanks for that Can I get screen recording demonstrating the issue

[ggazzo]

@schourasia750 can you inform the version you are? I'm sure its solved

[schourasia750]

Screen.Recording.2026-03-06.131725.mp4

[schourasia750]

@ggazzo you are wright that it has been improved in the latest version before this on multiple click ** keeps inserting.
But now still on writing anything inside then using ** again is inserting multiple **.
Also the button doesn't show that its been pressed like I showed in the video it is supposed to be.

[Utkarsh09102004]

Hi, I was able to reproduce this issue on my local setup.

I’d like to work on this. While testing it, I noticed that the problem seems to happen when formatting is applied again inside text that is already formatted, which can leave duplicated markdown symbols and broken output.

I had one small question before I start working on the fix. Would you prefer that I fix only the toolbar/shortcut behavior that is causing this issue right now, or should I try to make the composer handle these cases more generally, even when the user manually types malformed or more complex markdown?

I’m asking because I can scope it in two ways:

1. a smaller fix that targets the current reproducible formatting action path
2. a broader fix that tries to normalize more kinds of broken markdown input as well

The first option would be more focused and safer, while the second would be more complete but also a larger change.

[harsh-791]

Hi! I've analyzed this issue and identified the main root causes.

Findings

1. Incorrect Markdown Patterns

In messageBoxFormatting.ts, the formatting patterns do not follow the standard Markdown syntax:

• Bold currently uses:
  *{{text}}*
  It should use:
  **{{text}}**

• Strikethrough currently uses:
  ~{{text}}~
  It should use:
  ~~{{text}}~~

Using the correct Markdown syntax should ensure consistent formatting behavior.

---

2. Toggle Detection is Too Strict

The wrapSelection function currently detects formatting only when the Markdown symbols are immediately adjacent to the selection boundaries.

Because of this:

• It does not detect formatting when the cursor is inside already formatted text
• The formatting button inserts duplicate symbols instead of toggling the format

---

Proposed Fix

1. Correct the Markdown patterns in messageBoxFormatting.ts:

   • Use ** for bold
   • Use ~~ for strikethrough

2. Improve the wrapSelection detection logic:

   • Detect formatting by checking the surrounding context, not only the selection boundaries
   • Allow toggling even when the cursor is inside formatted text

3. (Optional enhancement) Add a visual active state to formatting buttons when the cursor is inside formatted text.

---

Expected Outcome

This should resolve both Issue #1 (button toggle behavior) and Issue #2 (duplicate symbols) from the bug report. Happy to implement this!

---

If this approach looks good, I'd be happy to implement the fix.

[ggazzo]

@ggazzo you are wright that it has been improved in the latest version before this on multiple click ** keeps inserting. But now still on writing anything inside then using ** again is inserting multiple **. Also the button doesn't show that its been pressed like I showed in the video it is supposed to be.

although the issue of the buttons being marked is true (and someone can fix it), the rest is intentional and we decided that the marking should be done with a character, if you want to update the issue, if not I'll end up closing it, as it doesn't seem like a problem to me