Closed
Bug 1483553
Opened 6 years ago
Closed 6 years ago
Medium.com's undo/redo is not working on FF Nightly for 2 months
Categories
(Web Compatibility :: Desktop, defect, P1)
Tracking
(firefox61 unaffected, firefox62 unaffected, firefox63 disabled, firefox64 disabled, firefox65 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox61 | --- | unaffected |
firefox62 | --- | unaffected |
firefox63 | --- | disabled |
firefox64 | --- | disabled |
firefox65 | --- | fixed |
People
(Reporter: dumindu.madunuwan, Assigned: adamopenweb)
References
Details
(Keywords: regression, site-compat)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:63.0) Gecko/20100101 Firefox/63.0
Build ID: 20180815100249
Steps to reproduce:
Related with https://www.reddit.com/r/firefox/comments/92j1ob/mediumcoms_undoredo_is_not_working_on_ff_nightly/ ,
* Create a new story on Medium.com using FF Nightly
* Make some changes like add inline code blocks or few words
* test undo/ redo functionality via key board shortcuts
Actual results:
Undo/ redo works only for last word change, sometimes totally not working as well.
Expected results:
undo/ redo functionality should work on nightly
Comment hidden (obsolete) |
Comment 2•6 years ago
|
||
This is due to Bug 1440189. Flipping the dom.keyboardevent.keypress.dispatch_non_printable_keys_only_system_group_in_content pref or adding `medium.com/p/` to the dom.keyboardevent.keypress.hack.dispatch_non_printable_keys pref works around the issue.
Comment 3•6 years ago
|
||
cc'ing astevenson, in case we have some contacts at Medium and they can fix it on their side like we did with GDocs?
Flags: needinfo?(astevenson)
Reporter | ||
Comment 4•6 years ago
|
||
https://help.medium.com/hc/requests/213276 for reference
Assignee | ||
Comment 5•6 years ago
|
||
I don't have a contact at Medium unfortunately, but have sent a message to Medium support as well.
Flags: needinfo?(astevenson)
Assignee | ||
Comment 6•6 years ago
|
||
Was contacted back by a Medium engineer today, provided the details of this report.
Comment 7•6 years ago
|
||
Just in case, we have a site compat doc for this issue: https://www.fxsitecompat.com/en-CA/docs/2018/non-printable-keys-will-soon-stop-firing-keypress-event/
Has Regression Range: --- → yes
Keywords: regression,
site-compat
Assignee | ||
Comment 8•6 years ago
|
||
awesome |
Thanks Kohei, I actually shared that link with them already. :)
Reporter | ||
Comment 10•6 years ago
|
||
Still having same issue btw
Assignee | ||
Comment 11•6 years ago
|
||
They are looking into the issue and have been active in communicating back with us. Will update when I have more info to share.
Flags: needinfo?(astevenson)
Updated•6 years ago
|
Flags: needinfo?(masayuki)
Updated•6 years ago
|
status-firefox64:
--- → affected
Comment 12•6 years ago
|
||
:kohei, can we close this now that Bug 1440189 is fixed? It doesn't reproduce on Nightly any longer at least.
Flags: needinfo?(kohei.yoshino)
Comment 13•6 years ago
|
||
No, it’s a regression introduced by Bug 1440189. I can still produce the issue. If it takes time for Medium to fix the issue, we have to add an exception like Etherpad and others. Otherwise, this bug should fall under Tech Evangelism.
Flags: needinfo?(kohei.yoshino)
Reporter | ||
Comment 14•6 years ago
|
||
As Kohei informed, this is still reproducible in Nightly and I faced this issue at least for 2 months in Nightly now. It's so grateful if Mozilla can fix this from browser side. Thanks
Comment 15•6 years ago
|
||
According to comment #2 and comment #13, could you take a look? Thank you so much.
Flags: needinfo?(masayuki)
Priority: -- → P3
Comment 16•6 years ago
|
||
This needs to be fixed from the Medium end. Bug 1440189 simply brings Gecko into line with the spec and other engines. We are not shipping it to release, however, until major Web compat issues such as this are resolved.
Flags: needinfo?(masayuki)
Comment 17•6 years ago
|
||
Yes, this needs to be fixed by Medium. On the other hand, we should add medium.com into the blacklist for avoiding inconvenience of Nightly testers (I thought that they'd fix the bug quickly).
Updated•6 years ago
|
Assignee: nobody → astevenson
Status: NEW → ASSIGNED
Component: DOM: Events → Desktop
Priority: P3 → --
Product: Core → Tech Evangelism
Version: 63 Branch → Firefox 63
Comment 18•6 years ago
|
||
I landed the patch for bug 1495627 to disable the new behavior only under "medium.com/p/". Therefore, staff of Medium will be not able to reproduce this bug with tomorrow's Nightly and newer. So, could you let them know that they need to remove ",medium.com/p/" from "dom.keyboardevent.keypress.hack.dispatch_non_printable_keys" in about:config.
Flags: needinfo?(astevenson)
Reporter | ||
Comment 20•6 years ago
|
||
Thanks for everyone for checking this and adding a patch from Nightly side. undo/ redo functionalities are now working perfectly on Nightly.
Updated•6 years ago
|
Priority: -- → P1
Comment 21•6 years ago
|
||
Looks like that undo/redo work fine with Nightly even if medium.com is removed from the blacklist. Did you get some replay from them?
Flags: needinfo?(astevenson)
Assignee | ||
Comment 22•6 years ago
|
||
I know they were working on a patch, but hadn't heard back. Just pinged them again. Thanks for the follow up. :)
Flags: needinfo?(astevenson)
Assignee | ||
Comment 23•6 years ago
|
||
They've confirmed that they fixed this issue. Closing this report as Fixed.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment 24•6 years ago
|
||
Thanks, I'll remove medium.com from the blacklist.
Updated•6 years ago
|
status-firefox65:
--- → fixed
Updated•6 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•