Skip to content

Add checkbox craftd compose#23

Merged
MatheusAraujoPro merged 12 commits intomainfrom
add-checkbox-craftd-compose
May 3, 2024
Merged

Add checkbox craftd compose#23
MatheusAraujoPro merged 12 commits intomainfrom
add-checkbox-craftd-compose

Conversation

@MatheusAraujoPro
Copy link
Copy Markdown
Collaborator

Description

In this PR i created the Checkbox compose component and did some changes on addBuilder method

Tests

I need some help to undersand this part 😂😂

Screenshots

image

Check list

  • The tests were executed and passed successfully.
  • [ x ] Code changes follow the project style guidelines.
  • Tests have been added if applicable.
  • [ x ] Did you subscribe to the channel?😛

Related Issues

[Add issues related to this PR, if any.]

  • [issue 1 link]

modifier = modifier,
)
if (checkboxProperties.hasItRightText == false)
Text(text = checkboxProperties.text ?: "empty")
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.

same thing

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I´ve done

vararg dynamicComposeBuilder: CraftDComposeBuilder
): CraftDComposeBuilders {
customBuilders.add(dynamicComposeBuilder)
customBuilders.addAll(dynamicComposeBuilder)
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.

sorry, I did yesterday , can you see that I did in another PR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I´ve done

@MatheusAraujoPro MatheusAraujoPro merged commit e897123 into main May 3, 2024
@MatheusAraujoPro MatheusAraujoPro deleted the add-checkbox-craftd-compose branch May 3, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants