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

Feat(web): Extend Flex direction by reverse value #1842

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

pavelklibani
Copy link
Contributor

@pavelklibani pavelklibani commented Jan 10, 2025

Description

  • Deprecate row and column values in direction prop.
  • Change Flex direction values to horizontal and vertical instead of row and column.
  • Extend Flex direction by horizontalReverse value.
  • Created codemod for value changes.
  • All demos updated to use new values and created new with reverse order.

Note

Only horizontal-reverse was implemented right now.
The vertical-reverse will be implemented in the future #DS-1630.

Additional context

The depreciation was due to the straightening of the dictionary values

Issue reference

Extend Flex direction

@pavelklibani pavelklibani self-assigned this Jan 10, 2025
@github-actions github-actions bot added the feature New feature or request label Jan 10, 2025
Copy link

netlify bot commented Jan 10, 2025

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit fc3c009
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/67817ca41beb8a0008442a68

Copy link

netlify bot commented Jan 10, 2025

Deploy Preview for spirit-design-system ready!

Name Link
🔨 Latest commit fc3c009
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system/deploys/67817ca467848c00089b75a0
😎 Deploy Preview https://deploy-preview-1842--spirit-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 96 (no change from production)
Accessibility: 91 (no change from production)
Best Practices: 100 (no change from production)
SEO: 91 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

- Deprecate row and column values
- Change Flex direction values to horizontal and vertical instead of row and column
- Extend Flex direction by horizontalReverse value
- All demos updated to use new values and created new with reverse order
- Deprecate row and column values
- Change Flex direction values to horizontal and vertical instead of row and column
- Extend Flex direction by horizontalReverse value
- All demos updated to use new values and created new with reverse order
@pavelklibani pavelklibani force-pushed the feat/ds-1607-flex-extended branch from ed0f6c9 to 456cd13 Compare January 10, 2025 08:44
@pavelklibani pavelklibani marked this pull request as ready for review January 10, 2025 09:54
@literat
Copy link
Collaborator

literat commented Jan 10, 2025

The vertical-reverse will be implemented in the future.

Can you link or create and link an issue for this, please?

@literat
Copy link
Collaborator

literat commented Jan 10, 2025

Please, checkout the markdown linting issues and fix them. Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants