Hi there, it seems to be an issue with last update. I tried to upload a video but it seems uploading is broken too.
When I try to use buttons on editor (or upload) in Firefox I see some errors on console regarding to Discourse chat:
I disabled all extensions and stills not working. Tell me if I can help you with something more as user
1 个赞
sam
(Sam Saffron)
2022 年6 月 30 日 07:50
2
Odd, is this still happening now?
Is it only related to videos, can you repro with images?
Here we go: https://ufile.io/z1tybnhf
[MP4 ~5MB file]
sam
(Sam Saffron)
2022 年7 月 1 日 04:48
6
Hmmm I am unable to repro with that file? (using Firefox on Linux)
Well, you need to download the MP4, read the first image or reproduce trying to bold your text in Firefox and Discourse.
Falco
(Falco)
2022 年7 月 2 日 00:04
8
This is a bold text with keyboard shortcut
This is a bold text with the menu click
Nope, no repro in Firefox for me.
link broken in firefox lol
I cant repo on firefox on windows 10.
sam
(Sam Saffron)
2022 年7 月 2 日 06:40
10
Yeah you need the exact right codecs, removed
1 个赞
That’s a bold tried with buttons.
That’s a bold tried with keyboard.
Sorry but I’m still with the same (without extensions).
Well I recorded with Kazam, but it’s OK:
2 个赞
sam
(Sam Saffron)
2022 年7 月 7 日 06:02
13
No luck reproducing here, just tried on firefox on linux. Perhaps try the browser safe mode ? Maybe it is browser plugin related.
Regarding that error, it looks somewhat unrelated and I think we fixed it a few days ago.
1 个赞
gerhard
(Gerhard Schlager)
2022 年7 月 7 日 09:09
14
What version of Firefox are you using? Are you using an ESR release?
2 个赞
78.15 ESR, what’s up with that?
Still not working in safe-mode.
sam
(Sam Saffron)
2022 年7 月 11 日 06:57
16
I am not sure if we even had 78.15 in our test matrix.
I think we need to improve our page here to at least call out stuff a bit better:
Depending on one’s interpretation Firefox ESR is either supported or not. We need to make a proper call here. (if unsupported we should display the HTML view)
My guess here @david is that this may be related to the undo support?
1 个赞
Falco
(Falco)
2022 年7 月 11 日 17:44
18
satonotdead:
78.15 ESR
Firefox 78 ESR support ended last year from Mozilla. It doesn’t receive security patches anymore. You need to update to the current ESR version (102) or at least the previous one (91) which will be supported for two more months.
3 个赞
david
(David Taylor)
2022 年7 月 11 日 19:50
19
I just double checked via BrowserStack, and the toolbar formatting buttons work correctly under Firefox 91
Yeah, via MDN , Firefox < 89 had bugs with text manipulation:
Before Firefox 89, manipulating the content of <input>
and <textarea>
elements using Document.execCommand()
commands requires workarounds (see bug 1220696 ).
4 个赞
Falco
(Falco)
2022 年7 月 11 日 19:52
20
We run CI on every commit on Firefox ESR. ESR version is the one packaged in the current Debian stable. At the moment it’s ran against 91.11.0
.
2 个赞
sam
(Sam Saffron)
2022 年7 月 11 日 21:12
21
Can we feature detect 89 and lower so it gets html view without impacting supported browsers?
1 个赞
david
(David Taylor)
2022 年7 月 12 日 09:38
22
It’s difficult to feature-detect a bug like this, so I think our only option is to check the user agent:
discourse:main
← discourse:old-browser-firefox-89
opened 09:28AM - 12 Jul 22 UTC
Prior to v89, Firefox has bugs with document.execCommand("insertText"): https://… bugzil.la/1220696
This commit introduces some variables to browser-detect, and therefore wraps the entire logic in an IIFE to avoid state leaking. (`let`/`const` are not supported on older browsers)
(best reviewed [with whitespace changes hidden](https://github.com/discourse/discourse/pull/17443/files?diff=split&w=1))
3 个赞
david
(David Taylor)
关闭
2022 年8 月 11 日 09:39
23
This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.