-
Notifications
You must be signed in to change notification settings - Fork 974
Remove CommonFormSection from widevineInfo.js #8911
Conversation
as widevineInfo is also used outside of widevinePanel.js Fixes #8907 Auditors: Test Plan 1: 1. Open about:preferences#plugins 2. Make sure there is margin between the divs, the div and the switch Test Plan 2: 1. Open netflix.com 2. Make sure the dialog design is not affected by this change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as per test plan. Good to go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
@Jacalz What's up with the milestone? |
What the ****, I have not even touched the milestones, might be time to change password 😅 Strange 😬 |
The strange thing is that only my devices are connected in the security log, but I went and enabled two factor authentication to be sure 😉😬 Sorry for what happened 🤦♂️ |
np! |
I had removed the milestone earlier and also I updated your comment above @Jacalz (let's keep in clean in here please 😄 ). Maybe your phone was unlocked in your pocket while viewing this page? |
Sorry, got really confused and frustrated 🙂 Have never happened before but let's say that it was that unless it happens again 😅 I am very sorry and actually glad that it wasn't anything huge that changed 🙂 |
as widevineInfo is also used outside of widevinePanel.js
Fixes #8907
Auditors:
Test Plan 1:
Test Plan 2:
Submitter Checklist:
git rebase -i
to squash commits (if needed).Reviewer Checklist:
Tests