Comments (4)
marijnh
commented on August 15, 2024
Oh, that's disappointing — they didn't implement their own spec, which explicitly says inverted selections are supported, properly.
I guess I'll push an emergency patch to turn EditContext integration off for the time being until we sort this out. Thanks for reporting these.
from dev.
jclem
commented on August 15, 2024
Thanks! Note that the MDN docs do mention that this is an exception: https://developer.mozilla.org/en-US/docs/Web/API/EditContext/updateSelection#exceptions
If start is greater than end, a DOMException is thrown.
(Although the MDN docs != the spec of course)
from dev.
marijnh
commented on August 15, 2024
This is fixed in Chrome 126. I've pushed a patch that disables EditContext use for older browser versions.
from dev.
jclem
commented on August 15, 2024
@marijnh Thank you! This was in Electron, which I guess is why I ran into it. Also explains why I couldn't reproduce it with plain old Vite in Chrome.
from dev.
Related Issues (20)
- Unable to style the complete merge view line HOT 6
- Conditionally-rendered block replace decoration breaks selection rendering and gutter HOT 9
- Selection doesn't drag vertically when text is too long horizontally HOT 2
- Incorrect view height in merge view with two same ~100k line documents HOT 5
- Mac OS native chinese IME, prompt pos not right. HOT 4
- UnifiedMergeView sometimes renders only the last character of a deleted line HOT 4
- Setting `EditorState.readOnly.of(true)` does not work anymore in Chrome since `v6.28.0` (EditContext API issue?) HOT 2
- Can't make editor read-only and not editable HOT 2
- line-height + lineWrapping causes inconsistent cursor positioning on click HOT 2
- Editor does not properly lose focus, causing other input fields in the browser to become unusable HOT 3
- Spell check fixing broken as of 6.28.0+ HOT 2
- LanguageTool and Grammarly cannot apply suggestions in Chrome HOT 2
- Markdown link text part parsing not working correctly HOT 1
- MergeView matching regression (example) HOT 2
- Suggestion: Support true inline diff in unified merge view
- Add number of results for the search query onto the search panel HOT 1
- Typing cursor out of sync on slow/proxied/domain environments HOT 4
- Autocompletion popup is not displayed if the start of the completion range is not visible HOT 1
- Cannot hover over the content of a hover tooltip when the arrow option is set HOT 2
- Can't create new lines in Stylus Legacy Mode
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dev.