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

refactor(action-group)!: remove deprecated layout property #10792

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

eriklharper
Copy link
Contributor

@eriklharper eriklharper commented Nov 19, 2024

BEGIN_COMMIT_OVERRIDE
refactor(action-group)!: remove deprecated layout property

Related Issue: #9713

Summary

BREAKING CHANGE: Removes the deprecated layout property by converting it to an internal property.

Developers will need to replace calcite-action-group's layout property with it's parent's layout property (i.e., parent action-pad or action-bar).
END_COMMIT_OVERRIDE

@github-actions github-actions bot added the docs Issues relating to documentation updates only. label Nov 19, 2024
@eriklharper eriklharper requested a review from jcfranco November 19, 2024 21:23
@eriklharper eriklharper changed the title docs(action-group): remove deprecated layout property by making it internal docs(action-group): remove deprecated layout property Nov 19, 2024
@eriklharper eriklharper changed the title docs(action-group): remove deprecated layout property docs(action-group)!: remove deprecated layout property Nov 19, 2024
@jcfranco
Copy link
Member

@eriklharper Can you use the refactor type? This will affect types as well.

@eriklharper eriklharper changed the title docs(action-group)!: remove deprecated layout property refactor(action-group)!: remove deprecated layout property Nov 20, 2024
@eriklharper eriklharper requested a review from jcfranco November 20, 2024 00:31
@github-actions github-actions bot added the refactor Issues tied to code that needs to be significantly reworked. label Nov 20, 2024
@jcfranco jcfranco added the skip visual snapshots Pull requests that do not need visual regression testing. label Nov 20, 2024
@eriklharper eriklharper merged commit a4d8538 into dev Nov 20, 2024
15 checks passed
@eriklharper eriklharper deleted the eriklharper/9173-action-group branch November 20, 2024 18:07
benelan pushed a commit that referenced this pull request Feb 8, 2025
**Related Issue:** #9173

## Summary

BREAKING CHANGE: Removes the deprecated `layout` property by converting
it to an internal property.

Developers will need to replace `calcite-action-group`'s `layout`
property with it's parent's `layout` property (i.e., parent `action-pad`
or `action-bar`).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issues relating to documentation updates only. refactor Issues tied to code that needs to be significantly reworked. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants