-
Notifications
You must be signed in to change notification settings - Fork 306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Large chunks of text (1000+) cannot not be copied in CodeMirror #225
Comments
Copying is handled by CodeMirror. See if their demo works for you https://codemirror.net and if not, report the bug in their repo. |
The full screen demo has more than 1000 characters and copies correctly. This appears to be a problem with Stylus, or at least the version of CodeMirror it uses. |
Yep, it was fixed in 5.28.0, but we use 5.27.4. |
@schomery, what about a maintenance release 1.1.4.3 on AMO to include several FF-related workarounds/fixes? If you agree, pull the locales and update manifest.json, please. |
@schomery has gone dark again. Starring in those Dos Equis commercials really keeps him busy. He had said he wanted to transfer the AMO account as well, but he's still the only one with access to it currently. FF 57 must've hit beta recently and the amount of illiterate, "tech-savvy" FF users is mind boggling. Many don't seem capable of reading or comprehending the disclaimer regarding UI styling, and appear to be completely oblivious when it comes to Mozilla's move to WebExtensions-only and the limitations which go along with it. I'm not claiming to always read descriptions either, but when something doesn't work as expected, I go back and read everything before unleashing my autism on the world. AFAICT, 57 is scheduled to hit stable in about a week. Not only do we need to release 1.1.4.3 because of all the FF-related workarounds/fixes, but we need to update the description so that the disclaimer is more prominent. At the very least, it should be all caps in the first couple sentences, especially since AMO recently began obscuring all text past the first 8 sentences. If they allow markdown, it should also be bold and red or something. Hell, maybe we should even put it in the title. I emailed schomery. Hopefully something gets his attention sometime soon. Edit: Looking at US.o just now, the recent surge in user feedback is likely due to the fact that SimilarWeb released a new FF version, which is supposed to be a DB importer for the legacy Stylish, in preparation for the actual full update to come soon after. The import does not appear to be going smoothly for everybody, so I guess more users are exploring their options. Another reason to update ASAP. |
sorry guys. I had no access to my email. @tophf should I pack from the current master? |
@narcolepticinsomniac and @tophf the AMO account is moved to the Gmail account so you guys also have access to it. I'll move the Opera version too if possible. |
Yeah, just pull dem da locales. |
done. |
I've updated release info. |
@schomery, no one spotted that Firefox options don't open now - I've fixed it, please reupload as |
Done |
Stylus 1.1.4.2 in Firefox ESR (52.3.0) on Fedora
When trying to copy a selection of more than 1000 characters from CodeMirror, the selection instead contains only a single hyphen. This happens in any CodeMirror box, such as the editor, import, and export text boxes.
The text was updated successfully, but these errors were encountered: