Closed Bug 1297239 Opened 8 years ago Closed 8 years ago

Enable eslint for browser/components/shell

Categories

(Firefox :: Shell Integration, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 51
Tracking Status
firefox51 --- fixed

People

(Reporter: jaws, Assigned: jaws)

References

Details

Attachments

(1 file)

      No description provided.
Comment on attachment 8783767 [details]
Bug 1297239 - Enable eslint for browser/components/shell.

https://reviewboard.mozilla.org/r/73454/#review71358

Thanks for doing this!

::: browser/components/shell/content/setDesktopBackground.js:2
(Diff revision 1)
> -# License, v. 2.0. If a copy of the MPL was not distributed with this
> -# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> +* License, v. 2.0. If a copy of the MPL was not distributed with this file,
> +* You can obtain one at http://mozilla.org/MPL/2.0/. */

Nit: line up the `*` please.

::: browser/components/shell/content/setDesktopBackground.js:140
(Diff revision 1)
> -        var y = (this._screenHeight - height) / 2;
> +        let y = (this._screenHeight - height) / 2;
>          ctx.drawImage(this._image, x, y, width, height);
>          break;
> -      case "FIT":
> +      }
> +      case "FIT": {
>          //Try maxing width first, top and bottom borders

Nit: update this comment with a space before and . after while we're here?
Attachment #8783767 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aac36e73b97a
Enable eslint for browser/components/shell. r=Gijs
https://hg.mozilla.org/mozilla-central/rev/aac36e73b97a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: