Rocket.Chat icon indicating copy to clipboard operation
Rocket.Chat copied to clipboard

Markdown hyperlink syntax

Open Zzzenk opened this issue 6 months ago • 10 comments

Search before asking

  • [X] I had searched in the issues and found no similar issues.

Operating System

  • [x] macOS
  • [X] Windows
  • [X] Linux

Operating System Version

No response

It happens on the web browser too?

Yes, it happens on the web browser too

Rocket.Chat Desktop App Version

3.9.11

Rocket.Chat Server Version

6.4.5

Describe the bug

If to type between two hyperlink any text in square brackets it's broke markdown.

Example:

## [v2023.12.21.3](https://https://github.com/)
- [BUG #11111](https://https://github.com/) - [BACK] changelog description
- [BUG #22222](https://github.com/) - changelog description
- [BUG #33333](https://https://github.com/) - [BACK] changelog description
- [BUG #44444](https://github.com/) - changelog description

image

How to Reproduce

  1. Copy and paste followng text to any chat:
## [v2023.12.21.3](https://https://github.com/)
- [BUG #11111](https://https://github.com/) - [BACK] changelog description
- [BUG #22222](https://github.com/) - changelog description
- [BUG #33333](https://https://github.com/) - [BACK] changelog description
- [BUG #44444](https://github.com/) - changelog description
  1. See broken markdown

Describe your Expected behavior

No response

Anything else

No response

Are you willing to submit a code contribution?

  • [ ] Yes, I am willing to submit a Pull Request!

Zzzenk avatar Dec 21 '23 08:12 Zzzenk

Hey @jeanfbrito, I see this as a major flaw in both the user experience and system. According to me, it's an important fix to make in the upcoming releases of the platform.

I would love to be assigned to work on this issue.

Thanks for the support, Prisha

preeesha avatar Dec 27 '23 18:12 preeesha

You dont need to be assigned to work on this, I will be waiting for your PR.

On Wed, Dec 27, 2023 at 3:13 PM Prisha Gupta @.***> wrote:

Hey @jeanfbrito https://github.com/jeanfbrito, I see its importance as a major flaw in both the user experience and system.

I would love to be assigned to work on this issue.

Thanks for the support Prisha

— Reply to this email directly, view it on GitHub https://github.com/RocketChat/Rocket.Chat/issues/31766, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI7PCRKBPOAUHCSEML2G3DYLRQLNAVCNFSM6AAAAABA573GNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGUZDKNBTGA . You are receiving this because you were mentioned.Message ID: @.***>

-- Atenciosamente, Jean Brito.

jeanfbrito avatar Dec 28 '23 01:12 jeanfbrito

Okay, thanks for the highlight. I am starting to working on this now. Will shortly update you with the progress.

preeesha avatar Dec 28 '23 05:12 preeesha

Hi @jeanfbrito,

I've looked into the issue, and it seems like a bug in the Rocket.Chat's web view's rendering system rather than of the electron app. The problem lies in the code responsible for rendering the messaging web view.

Screenshots:

  1. Here's the code that handles the rendering of Rocket.Chat's messaging web view image
  2. DOM rendering webview image

To fix this, it needs attention in the Rocket.Chat (https://github.com/RocketChat/Rocket.Chat) repository.

Injecting temporary CSS or JavaScript here isn't a good long-term solution.

Please consider escalating this to the Rocket.Chat repository for a proper fix. Let me know if you need more info or help.

Thanks, Prisha Gupta

preeesha avatar Dec 30 '23 17:12 preeesha

haha, nice AI response it was @preeesha , any it doesn't solve the issue its only need a small JS fix.

aryanpnd avatar Dec 31 '23 01:12 aryanpnd

Ohhh I see the humour rolling @aryanpnd. It's sad to see people getting confused between human and AI responses nowadays. After all, it's the age of AI, I get you it's not your mistake.

Nonetheless, rather than commenting on stuff that doesn't solve the issue, you should just solve the issue. It's highly advisable to maintain professionalism here.

I hope you get the point. And I wish you good luck for your small JS fixes.

Thank you

preeesha avatar Dec 31 '23 05:12 preeesha

Fixed issue RocketChat/Rocket.Chat#31766: Resolved a problem in the Fuselage repository with this pull request: https://github.com/RocketChat/fuselage/pull/1260

Giridharan002 avatar Jan 10 '24 19:01 Giridharan002

@jeanfbrito hi! I saw v3.9.13 was released yesterday, but there is no fix RocketChat/Rocket.Chat#31766. Will it be added in 3.9.14?

Zzzenk avatar Feb 13 '24 07:02 Zzzenk

I'm sorry, I closed by mistake. This is an issue with the core, so I will move it to there. It's not the desktop app that does that.

jeanfbrito avatar Feb 15 '24 20:02 jeanfbrito

@dougfabris can you take a look? Seems we have the fix already.

jeanfbrito avatar Feb 15 '24 20:02 jeanfbrito

@jeanfbrito hi! Do you know when this fix will be released? I'm really waiting for it))

Zzzenk avatar Feb 21 '24 10:02 Zzzenk

@jeanfbrito hello?

Zzzenk avatar Feb 26 '24 08:02 Zzzenk

Anybody?

Zzzenk avatar Mar 13 '24 09:03 Zzzenk

"Awesome" support... Total ignore...

Zzzenk avatar Apr 03 '24 18:04 Zzzenk