Revert "Add namespaced styles to primers spacing (#2962)"

[siddharthkp]

• This reverts commit 5a16f92.
• Decided that this change is for safer migration in dotcom and better off just in github/github-ui instead of primer/css: https://github.com/github/github-ui/pull/12160

[changeset-bot]

⚠️ No Changeset found

Latest commit: baed8e6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Copilot]

Pull request overview

This PR reverts a previous change that added pr- namespaced utility classes for margin and padding. The decision was made to keep these namespaced styles in github/github-ui rather than in primer/css for safer migration in dotcom.

Changes:

• Removes all pr- prefixed variants of margin utility classes (.pr-m*, .pr-mt*, .pr-mb*, .pr-mr*, .pr-ml*, .pr-mx*, .pr-my*)
• Removes all pr- prefixed variants of padding utility classes (.pr-p*, .pr-pt*, .pr-pr*, .pr-pb*, .pr-pl*, .pr-px*, .pr-py*)
• Removes the associated changeset file documenting the namespace addition

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
src/utilities/padding.scss	Removes pr- namespaced variants of all padding utility classes, reverting to single-line formatting for most class definitions
src/utilities/margin.scss	Removes pr- namespaced variants of all margin utility classes (including negative margins and auto margins), reverting to single-line formatting for most class definitions
.changeset/empty-islands-push.md	Deletes the changeset that documented the addition of the pr- namespace for spacing utilities

Comments suppressed due to low confidence (1)

src/utilities/margin.scss:36

• There's extra whitespace before the colon in the property declaration. The property should be margin-left: (without the extra space) for consistency with the rest of the codebase.

          .ml#{$variant}-n#{$scale} { margin-left  : calc(-1 * $size) !important; }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

There are two spaces between the selector and the opening brace. This should be a single space for consistency with the rest of the codebase (see other lines in this file and src/utilities/borders.scss which use single spaces).

Suggested change

	.m#{$variant}-#{$scale} { margin: $size !important; }
	.m#{$variant}-#{$scale} { margin: $size !important; }

[Copilot]

There's extra whitespace before the colon in the property declaration. The property should be margin-right: (without the extra space) for consistency with the rest of the codebase.

Suggested change

	.mr#{$variant}-n#{$scale} { margin-right : calc(-1 * $size) !important; }
	.mr#{$variant}-n#{$scale} { margin-right: calc(-1 * $size) !important; }

[Copilot]

There are two spaces between the selector and the opening brace. This should be a single space for consistency with the rest of the codebase (see other lines in this file and src/utilities/borders.scss which use single spaces).

Suggested change

	.p#{$variant}-#{$scale} { padding: $size !important; }
	.p#{$variant}-#{$scale} { padding: $size !important; }