Skip to content

Commit

Permalink
Bug 1751899 - [devtools] Remove compatibility panel Feedback button. …
Browse files Browse the repository at this point in the history
…r=Honza.

Differential Revision: https://phabricator.services.mozilla.com/D136996
  • Loading branch information
nchevobbe committed Jan 26, 2022
1 parent 3e39889 commit c72e916
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 48 deletions.
17 changes: 0 additions & 17 deletions devtools/client/inspector/compatibility/components/Footer.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,6 @@ const {
updateSettingsVisibility,
} = require("devtools/client/inspector/compatibility/actions/compatibility");

loader.lazyRequireGetter(
this,
"openDocLink",
"devtools/client/shared/link",
true
);

const FEEDBACK_LINK =
"https://docs.google.com/forms/d/e/1FAIpQLSeevOHveQ1tDuKYY5Fxyb3vvbKKumdLWUT5-RuwJWoAAOST5g/viewform";

const REPORT_ICON = "chrome://devtools/skin/images/report.svg";
const SETTINGS_ICON = "chrome://devtools/skin/images/settings.svg";

class Footer extends PureComponent {
Expand Down Expand Up @@ -80,12 +69,6 @@ class Footer extends PureComponent {
"compatibility-settings-button-label",
"compatibility-settings-button-title",
this.props.updateSettingsVisibility
),
this._renderButton(
REPORT_ICON,
"compatibility-feedback-button-label",
"compatibility-feedback-button-title",
() => openDocLink(FEEDBACK_LINK)
)
);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,33 +32,5 @@ exports[`Footer component renders 1`] = `
</Localized>
</button>
</Localized>
<Localized
attrs={
Object {
"title": true,
}
}
id="compatibility-feedback-button-title"
>
<button
className="compatibility-footer__button"
onClick={[Function]}
title="compatibility-feedback-button-title"
>
<img
className="compatibility-footer__icon"
src="chrome://devtools/skin/images/report.svg"
/>
<Localized
id="compatibility-feedback-button-label"
>
<label
className="compatibility-footer__label"
>
compatibility-feedback-button-label
</label>
</Localized>
</button>
</Localized>
</footer>
`;
3 changes: 0 additions & 3 deletions devtools/client/locales/en-US/compatibility.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ compatibility-issue-deprecated-experimental-prefixneeded = (deprecated, experime
compatibility-settings-button-label = Settings
compatibility-settings-button-title =
.title = Settings
compatibility-feedback-button-label = Feedback
compatibility-feedback-button-title =
.title = Feedback
## Messages used as headers in settings pane

Expand Down

0 comments on commit c72e916

Please sign in to comment.