Skip to content

MessageService - Add MessageAlignment option to MessageOptions#6645

Open
paulosdfigueiredo wants to merge 1 commit into
Megabit:masterfrom
paulosdfigueiredo:dev-message-alignment-in-message-service
Open

MessageService - Add MessageAlignment option to MessageOptions#6645
paulosdfigueiredo wants to merge 1 commit into
Megabit:masterfrom
paulosdfigueiredo:dev-message-alignment-in-message-service

Conversation

@paulosdfigueiredo

Copy link
Copy Markdown
Contributor

Description

This PR introduces a new MessageAlignment property to MessageOptions, allowing customization of message text alignment.
MessageProvider has been updated to respect this property when rendering messages. By default, messages are center-aligned to preserve existing behavior.
Additionally, the alignment logic has been refactored by introducing a GetMessageAlignment helper method, and the default value is now defined in MessageOptions.Default.

How Has This Been Tested?

Manually tested by rendering messages with different MessageAlignment values and verifying that the alignment is correctly applied.
Also confirmed that the default behavior remains unchanged (center alignment) when the property is not explicitly set.

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 not to work as expected)

Checklist:

  • The PR is submitted to the correct branch (master for dev, rel-1.x for maintenance).
  • My code follows the code style of this project.
  • I've added relevant tests.

Introduce MessageAlignment property to MessageOptions for customizable message text alignment. Update MessageProvider to use this property for ModalBody and Text elements, defaulting to center alignment. Refactor code to use GetMessageAlignment method and set default alignment in MessageOptions.Default.
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.

1 participant