Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert #3389 (PageLayout.Pane: Undo deprecation for position prop) #3589

Merged
merged 9 commits into from
Aug 4, 2023

Conversation

siddharthkp
Copy link
Member

@siddharthkp siddharthkp commented Aug 4, 2023

production (before) upgrade branch (after)
Sidebar in hyperlist is on the left Sidebar in hyperlist is on the left Sidebar in hyperlist is on the right Sidebar in hyperlist is on the right

We would need to bring this change back in a way that it allows new behaviour without breaking existing behaviour.

  • Sidenote: It seems like the same pull request number is also merged in next-major? Link to next-major release tracking That seems very fishy? Was this not supposed to be in main at all?

@siddharthkp siddharthkp requested review from a team and broccolinisoup August 4, 2023 14:26
@changeset-bot
Copy link

changeset-bot bot commented Aug 4, 2023

🦋 Changeset detected

Latest commit: 3c49ffd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/react Patch

Not sure what this means? Click here to learn what changesets are.

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

@siddharthkp siddharthkp requested review from joshblack and removed request for broccolinisoup August 4, 2023 14:27
@siddharthkp siddharthkp changed the title Revert #3389 (Deprecate position prop for PageLayout.Pane) Revert #3389 (PageLayout.Pane: Undo deprecation for position prop) Aug 4, 2023
@siddharthkp
Copy link
Member Author

siddharthkp commented Aug 4, 2023

FYI @radglob @broccolinisoup, sorry! :(

@siddharthkp siddharthkp self-assigned this Aug 4, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2023

size-limit report 📦

Path Size
dist/browser.esm.js 103.67 KB (-0.02% 🔽)
dist/browser.umd.js 104.21 KB (-0.01% 🔽)

@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 14:33 Inactive
@siddharthkp siddharthkp temporarily deployed to github-pages August 4, 2023 14:35 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 14:36 Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 14:36 Inactive
@siddharthkp siddharthkp temporarily deployed to github-pages August 4, 2023 14:43 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 14:44 Inactive
@siddharthkp siddharthkp added update snapshots patch release bug fixes, docs, housekeeping labels Aug 4, 2023
@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 15:06 Inactive
@siddharthkp siddharthkp temporarily deployed to github-pages August 4, 2023 15:09 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to storybook-preview-3589 August 4, 2023 15:10 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch release bug fixes, docs, housekeeping
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants