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

Added enhancements to our versions docs #2635

Merged
merged 3 commits into from
Feb 14, 2025
Merged

Conversation

js-goupil
Copy link
Contributor

@js-goupil js-goupil commented Feb 13, 2025

Background

Actioning on the feedback that we lacked documentation in the version changes for 2025.1 and POS 9.27.0

Solution

🎩

Checklist

  • I have 🎩'd these changes
  • I have updated relevant documentation

Comment on lines 90 to 101
- Added sizing and fill options to the [Image](/docs/api/pos-ui-extensions/components/image) component.
- Added sizing and fill options to the[Image](/docs/api/pos-ui-extensions/components/image) component.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why tho

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EMBARASSING


### Important fix in POS 9.27.0

- Fixed an issue where the \`Stack\` component was not rendering correctly. By default, \`Stack\` will now always wrap. Additionally, stacks using \`'block'\` will now compress horizontally. Use \`alignContents: 'stretch'\` to take all of the horizontal space. This will not have an impact on layout in previous POS versions.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When we change things like this, we should be providing screenshots of the diff. Trying to parse words is incredibly hard. Seeing the diff in images is very easy.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're gonna add a code example with a screenshot. I think when we do that, we can just link to that doc from here

@js-goupil js-goupil force-pushed the js-fix-versions-docs branch from 3984aaf to 229b22e Compare February 14, 2025 16:34
@@ -64,6 +61,14 @@ Refer to the [migration guide](/docs/api/pos-ui-extensions/migrating) for more i
- Removed in POS version: N/A
- Release day: 1/6/2025

### Important fix in POS 9.29.0

- Fixed an issue where the \`FormattedTextField\` were rendering black input text in dark mode. It will now render the correct light color variant.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Fixed an issue where the \`FormattedTextField\` were rendering black input text in dark mode. It will now render the correct light color variant.
- Fixed an issue where the \`FormattedTextField\` were rendering black input text in dark mode.

Do you think we need to explain the correct behaviour?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

were -> was, too, just noticed

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"where the `Stack` component was not"

so I'd add the word component for consistency with the other change below, too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

god bless you boys

@js-goupil js-goupil force-pushed the js-fix-versions-docs branch from 229b22e to 8340894 Compare February 14, 2025 18:41
@js-goupil js-goupil merged commit 4835263 into unstable Feb 14, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants