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

fix: allow MaxWidth enum instance #14996

Open
wants to merge 2 commits into
base: 4.x
Choose a base branch
from

Conversation

maartenpaauw
Copy link
Contributor

Description

The getMaxWidth() method is compatible with handling the MaxWidth enum. Previously, it was not possible to use MaxWidth enum instances when overwriting the $maxWidth property.

Note: This change was not applied to the 3.x branch as it introduces a breaking change. The change may impact classes extending SimplePage or EditProfile if the maxWidth property has already been overridden.

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

maartenpaauw and others added 2 commits December 4, 2024 12:33
The `getMaxWidth()` method is compatible with handling the `MaxWidth` enum. Previously, it was not possible to use `MaxWidth` enum instances when overwriting the `$maxWidth` property.

Note: This change was not applied to the `3.x` branch as it introduces a breaking change. The change may impact classes extending `SimplePage` or `EditProfile` if the `maxWidth` property has already been overridden.
@danharrin danharrin added this to the v4 milestone Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

2 participants