Closed Bug 1569574 Opened 8 months ago Closed 8 months ago

Rename responsive.html/ to responsive/ and format using Prettier


(DevTools :: Responsive Design Mode, task, P3)



(firefox70 fixed)

Firefox 70
Tracking Status
firefox70 --- fixed


(Reporter: pbro, Assigned: gl)




(2 files)

Prettier is now used throughout the mozilla-central repository to re-format JS files.
A few files and directories are still ignored for various reasons. In particular .html files are ignored. See

It is unfortunate that RDM lives in a folder that contains the .html suffix though:

This means that all JS files within this directory are not being re-formatted by prettier.

We should either change the .prettierignore file to include RDM, or rename responsive.html to just responsive, or rdm. The .html suffix is historical and doesn't really have any meaning today.

I think we should rename this to just responsive since we don't have any folder names with just the acronym of the panel.

All of the following places will need to be changed:

Also once we do this, then prettier will all-of-a-sudden re-format all files, so this may cause people to rebase their inflight patches.

Priority: -- → P3

For safety, let's do this as a three-part process:

  1. Add an exception to .prettierignore to add the responsive.html directory. Make sure this change is annotated such that our indexing tools understand that all the prettier changes are skipped for "blame".
  2. Do the actual renaming.
  3. Remove the exception from .pretterignore.

Hey @vporof, I am assuming a=automatic-formatting will cause the changeset to be ignored in our indexing tools and blame, however, let me know if we need to do more. Also, feel free to commandeer the patch if needed.

Flags: needinfo?(vporof)
Assignee: nobody → gl

You'll need to add # skip-this-changset to the commit message, and populate the list at and

Flags: needinfo?(vporof)

See bug 1561435 as an example of the commit message, and bug 1563763 and bug 1564862 for adding the blame changesets.

Summary: RDM has been accidentally ignored from prettier → Rename responsive.html/ to responsive/ and format using Prettier
Blocks: 1572125
Pushed by
Part 1: Rename the responsive.html/ folder to responsive/. r=pbro
Part 2: Format devtools/client/responsive/. a=automatic-formatting r=vporof
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.