Skip to content

Bug 2073095: fix text selection test - #380

Closed
pollym wants to merge 1 commit into
mozilla-firefox:autolandfrom
pollym:fix-text-selection-test
Closed

pollym wants to merge 1 commit into
mozilla-firefox:autolandfrom
pollym:fix-text-selection-test

Conversation

@pollym

@pollym pollym commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

There were subtle differences in behaviour based on what url is shown in the text selection bar. Putting in an extra step to ensure that we have grabbed all of the text here.
try is running here


Lando: link
Bugzilla: bug 2073095

🚫 This pull request has 2 blockers.

There were subtle differences in behaviour based on what url is shown in the text selection bar. Putting in an extra step to ensure that we have grabbed all of the text here.
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

View this pull request in Lando to land it once approved.

@segunfamisa segunfamisa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh nice find! I didn't come across this in the spike, but lovely to see.

Thank you Polly

@lando-worker

lando-worker Bot commented Oct 5, 2026

Copy link
Copy Markdown

Pull request closed by commit 5e5b216

lando-worker Bot pushed a commit that referenced this pull request Oct 5, 2026
There were subtle differences in behaviour based on what url is shown in the text selection bar. Putting in an extra step to ensure that we have grabbed all of the text here.
[try is running here](https://treeherder.mozilla.org/jobs?repo=try&landoInstance=lando-prod-2025&landoCommitID=100016)

Pull request: #380
@lando-worker lando-worker Bot closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants