Skip to content

[Editor] Make sure the comment dialog is visible on the screen (bug 1989304)#20283

Merged
calixteman merged 1 commit intomozilla:masterfrom
calixteman:bug1989304
Sep 23, 2025
Merged

[Editor] Make sure the comment dialog is visible on the screen (bug 1989304)#20283
calixteman merged 1 commit intomozilla:masterfrom
calixteman:bug1989304

Conversation

@calixteman
Copy link
Contributor

No description provided.

@calixteman
Copy link
Contributor Author

/botio integrationtest

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_integrationtest from @calixteman received. Current queue size: 0

Live output at: http://54.241.84.105:8877/965679e11b42fad/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Received

Command cmd_integrationtest from @calixteman received. Current queue size: 0

Live output at: http://54.193.163.58:8877/e921f8fe2b9b0ca/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/965679e11b42fad/output.txt

Total script time: 18.65 mins

  • Integration Tests: Passed

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Windows)


Success

Full output at http://54.193.163.58:8877/e921f8fe2b9b0ca/output.txt

Total script time: 44.09 mins

  • Integration Tests: Passed

@calixteman calixteman added the To Review For PRs to review after they've been merged (for example when reviewers are off). label Sep 23, 2025
@calixteman calixteman merged commit 225b07a into mozilla:master Sep 23, 2025
9 checks passed
@calixteman calixteman deleted the bug1989304 branch September 23, 2025 17:30
@timvandermeij timvandermeij removed their request for review October 4, 2025 12:21
@timvandermeij timvandermeij removed the To Review For PRs to review after they've been merged (for example when reviewers are off). label Oct 5, 2025
@timvandermeij
Copy link
Contributor

Looks good to me, and thank you for introducing the first commenting tests here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants