Bug 1949259 - Enforce valid HTML in HTML for Lit templates r=frontend-codestyle-reviewers,mtigley,credential-management-reviewers,fxview-reviewers,sidebar-reviewers,mossop
Differential Revision: https://phabricator.services.mozilla.com/D245591
This commit is contained in:
committed by
mkennedy@mozilla.com
parent
a7c081f8f6
commit
dc89961c6d
@@ -69,6 +69,7 @@ module.exports = {
|
||||
plugins: ["mozilla", "html", "import", "json", "promise", "lit"],
|
||||
rules: {
|
||||
"lit/quoted-expressions": ["error", "never"],
|
||||
"lit/no-invalid-html": "error",
|
||||
},
|
||||
overrides: [
|
||||
{
|
||||
|
||||
@@ -54,7 +54,7 @@ export default class Timeline extends MozLitElement {
|
||||
style=${styles}
|
||||
class="action"
|
||||
data-l10n-id=${entry.actionId}
|
||||
</div>
|
||||
></div>
|
||||
`;
|
||||
});
|
||||
|
||||
|
||||
@@ -52,11 +52,11 @@ class FxviewEmptyState extends MozLitElement {
|
||||
if (!descriptionLink) {
|
||||
return html``;
|
||||
}
|
||||
return html` <a
|
||||
return html`<a
|
||||
data-l10n-name=${descriptionLink.name}
|
||||
href=${descriptionLink.url}
|
||||
target=${descriptionLink?.sameTarget ? "_self" : "_blank"}
|
||||
/>`;
|
||||
></a>`;
|
||||
}
|
||||
|
||||
render() {
|
||||
|
||||
@@ -77,15 +77,19 @@ class ShoppingSettings extends MozLitElement {
|
||||
let canShowRecommendationToggle = this.adsEnabled;
|
||||
|
||||
let adsToggleMarkup = canShowRecommendationToggle
|
||||
? html`
|
||||
<div class="shopping-settings-toggle-option-wrapper">
|
||||
? html` <div class="shopping-settings-toggle-option-wrapper">
|
||||
<moz-toggle
|
||||
id="shopping-settings-recommendations-toggle"
|
||||
?pressed=${this.adsEnabledByUser}
|
||||
data-l10n-id="shopping-settings-recommendations-toggle2"
|
||||
data-l10n-attrs="label"
|
||||
@toggle=${this.onToggleRecommendations}>
|
||||
<span id="shopping-ads-learn-more" data-l10n-id="shopping-settings-recommendations-learn-more3" slot="description">
|
||||
@toggle=${this.onToggleRecommendations}
|
||||
>
|
||||
<span
|
||||
id="shopping-ads-learn-more"
|
||||
data-l10n-id="shopping-settings-recommendations-learn-more3"
|
||||
slot="description"
|
||||
>
|
||||
<a
|
||||
id="shopping-ads-learn-more-link"
|
||||
target="_blank"
|
||||
@@ -95,7 +99,7 @@ class ShoppingSettings extends MozLitElement {
|
||||
data-l10n-name="review-quality-url"
|
||||
></a>
|
||||
</span>
|
||||
</moz-toggle/>
|
||||
</moz-toggle>
|
||||
</div>`
|
||||
: null;
|
||||
|
||||
|
||||
@@ -69,7 +69,7 @@ class ShoppingCard extends MozLitElement {
|
||||
<span id="heading">${this.label}</span>
|
||||
<moz-five-star
|
||||
rating=${this.rating === 0 ? 0.5 : this.rating}
|
||||
</moz-five-star>
|
||||
></moz-five-star>
|
||||
</div>`;
|
||||
}
|
||||
return "";
|
||||
|
||||
@@ -181,7 +181,7 @@ export class SidebarCustomize extends SidebarPage {
|
||||
data-l10n-id=${this.getInputL10nId(tool.view)}
|
||||
@change=${this.onToggleToolInput}
|
||||
?checked=${!tool.disabled}
|
||||
/>
|
||||
></moz-checkbox>
|
||||
`;
|
||||
}
|
||||
|
||||
@@ -282,7 +282,7 @@ export class SidebarCustomize extends SidebarPage {
|
||||
?checked=${this.getWindow().SidebarController._state
|
||||
.revampVisibility === "expand-on-hover"}
|
||||
?disabled=${this.visibility == "hide-sidebar"}
|
||||
/>
|
||||
></moz-checkbox>
|
||||
`
|
||||
)}
|
||||
<moz-checkbox
|
||||
|
||||
@@ -179,7 +179,7 @@ class SyncedTabsInSidebar extends SidebarPage {
|
||||
.searchQuery=${this.controller.searchQuery}
|
||||
@fxview-tab-list-primary-action=${navigateToLink}
|
||||
@fxview-tab-list-secondary-action=${this.onSecondaryAction}
|
||||
/>
|
||||
></sidebar-tab-list>
|
||||
</moz-card>`;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user