Skip to content

Conversation

@khanjasir90
Copy link
Contributor

Description

  1. Added docs for Mirai Padding widget.

Related Issues

Closes #58

Type of Change

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

Copy link
Member

@divyanshub024 divyanshub024 left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @khanjasir90. Please check my suggestion.

| --- |-------------------|---------------------------------------------------|
| padding | `MiraiEdgeInsets` | The amount of space by which to inset the child. |
| child | `Map<String,dynamic>` | The widget below this widget in the tree. |

Copy link
Member

Choose a reason for hiding this comment

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

Hey @khanjasir90, In note or description can we also mention other ways to give padding?

  1. All padding: {"padding": 12}
  2. Padding fromLTRB: {"padding": [8, 12, 8, 12]}.

@divyanshub024 divyanshub024 changed the title feat: added mirai padding docs docs: added mirai padding docs Jan 9, 2025
@divyanshub024
Copy link
Member

Hey @khanjasir90, Any update on the above suggestions? :)

@khanjasir90
Copy link
Contributor Author

Hey @divyanshub024, just got busy with some work!, will make required changes by tomorrow.

@divyanshub024 divyanshub024 merged commit 57117e6 into StacDev:dev Jan 12, 2025
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.

docs: Add Mirai Padding Parser Docs

2 participants