Skip to content

refactor: Adds display property in Flex Component#65

Merged
pruthvi2103 merged 2 commits intoantstackio:mainfrom
Raalzz:fix-flex-component
Aug 18, 2022
Merged

refactor: Adds display property in Flex Component#65
pruthvi2103 merged 2 commits intoantstackio:mainfrom
Raalzz:fix-flex-component

Conversation

@Raalzz
Copy link
Copy Markdown
Member

@Raalzz Raalzz commented Aug 17, 2022

Description

  • Adds display property in Flex Component

Motivation and Context

How Has This Been Tested?

  • Tested Locally

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@netlify
Copy link
Copy Markdown

netlify bot commented Aug 17, 2022

Deploy Preview for hover-design ready!

Name Link
🔨 Latest commit bd05bf4
🔍 Latest deploy log https://app.netlify.com/sites/hover-design/deploys/62fcf2b9d601b70008b34bda
😎 Deploy Preview https://deploy-preview-65--hover-design.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

flexGrow?: React.CSSProperties["flexGrow"];
flexBasis?: React.CSSProperties["flexBasis"];
gap?: React.CSSProperties["gap"];
display?: "flex" | "inline-flex";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

okay okay !! I didn't know there was something inline-flex in the display property.
Got it now !!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cool then Merging, awesome seeing improvements on components we are using 😄

@pruthvi2103 pruthvi2103 merged commit cd44b4c into antstackio:main Aug 18, 2022
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