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
Fix incremental reflow of text selection changes #10347
Conversation
Why do we force relayout here instead of dirtying the node and doing a regular layout? |
force_relayout is from an older time. All it does is call document.content_changed(self), which just calls node.dirty(), so it's equivalent. |
Let's switch to explicit dirtying then. |
Done. |
Looks good, squash if you like and r=me |
What bug? |
Also replace force_relayout methods with direct Node::dirty calls, for clarity.
94655ea
to
776aae6
Compare
@bors-servo r=pcwalton
When the URL is auto-completed, the auto-inserted part is selected, but (sometimes?) Servo does not highlight the selection right away. |
📌 Commit 776aae6 has been approved by |
Fix incremental reflow of text selection changes r? @emilio or @pcwalton CC @paulrouget: This fixes a bug in the browser.html URL bar auto-completion. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10347) <!-- Reviewable:end -->
💔 Test failed - mac-rel-css |
Fix incremental reflow of text selection changes r? @emilio or @pcwalton CC @paulrouget: This fixes a bug in the browser.html URL bar auto-completion. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10347) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, status-appveyor |
r? @emilio or @pcwalton
CC @paulrouget: This fixes a bug in the browser.html URL bar auto-completion.
This change is