Skip to content
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

Change window title from "MetaMask Notification" to "MetaMask Dialog" #21680

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

eriknson
Copy link
Member

@eriknson eriknson commented Nov 3, 2023

Description

Changes the window title from MetaMask Notification to MetaMask Dialog

Manual testing steps

  1. Prompt any MetaMask window / popup
  2. Confirm that the window's title is "MetaMask Dialog"

@eriknson eriknson requested review from a team as code owners November 3, 2023 14:25
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Nov 3, 2023
@eriknson eriknson added team-extension-ux DEPRECATED: please use "team-wallet-ux" label instead and removed INVALID-PR-TEMPLATE PR's body doesn't match template labels Nov 3, 2023
@HowardBraham
Copy link
Contributor

This isn't passing the E2E tests, and I don't think it could. The problem is that the main window is also called "MetaMask." Why do you want to change the window title anyway?

@eriknson
Copy link
Member Author

eriknson commented Nov 6, 2023

This isn't passing the E2E tests, and I don't think it could. The problem is that the main window is also called "MetaMask."

I see, will try to figure this out.

Why do you want to change the window title anyway?

We're introducing notifications (natively and via snaps) so it'll be confusing if this is also called "Notification"

@eriknson eriknson requested review from a team, kumavis and brad-decker as code owners December 7, 2023 20:03
Copy link
Contributor

github-actions bot commented Dec 7, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@HowardBraham HowardBraham changed the title Change window title to "MetaMask" Change window title from "MetaMask Notifcation" to "MetaMask Dialog" Dec 7, 2023
@HowardBraham HowardBraham changed the title Change window title from "MetaMask Notifcation" to "MetaMask Dialog" Change window title from "MetaMask Notification" to "MetaMask Dialog" Dec 7, 2023
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Dec 7, 2023
Copy link

codecov bot commented Dec 7, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a6b592d) 67.74% compared to head (ad45cf5) 67.74%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #21680   +/-   ##
========================================
  Coverage    67.74%   67.74%           
========================================
  Files         1054     1054           
  Lines        40844    40844           
  Branches     10945    10945           
========================================
  Hits         27669    27669           
  Misses       13175    13175           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@metamaskbot
Copy link
Collaborator

Builds ready [ad45cf5]
Page Load Metrics (1346 ± 128 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint843041716933
domContentLoaded9219657838
load82717401346268128
domInteractive9219657838
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@HowardBraham HowardBraham self-assigned this Dec 7, 2023
@HowardBraham HowardBraham requested review from Gudahtt and bowensanders and removed request for a team December 7, 2023 23:01
@HowardBraham HowardBraham removed request for a team and kumavis December 7, 2023 23:01
Copy link
Contributor

@bowensanders bowensanders left a comment

Choose a reason for hiding this comment

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

LGTM! :)

@HowardBraham HowardBraham merged commit e8118f8 into develop Dec 7, 2023
72 checks passed
@HowardBraham HowardBraham deleted the en/title-change branch December 7, 2023 23:14
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2023
@metamaskbot metamaskbot added the release-11.9.0 Issue or pull request that will be included in release 11.9.0 label Dec 7, 2023
@chloeYue chloeYue added release-11.8.0 Issue or pull request that will be included in release 11.8.0 and removed release-11.9.0 Issue or pull request that will be included in release 11.9.0 labels Dec 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
INVALID-PR-TEMPLATE PR's body doesn't match template release-11.8.0 Issue or pull request that will be included in release 11.8.0 team-extension-ux DEPRECATED: please use "team-wallet-ux" label instead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants