Skip to content

Implement text centering#121

Merged
Miepee merged 2 commits intomainfrom
centered-text
Feb 13, 2025
Merged

Implement text centering#121
Miepee merged 2 commits intomainfrom
centered-text

Conversation

@biosp4rk
Copy link
Copy Markdown
Collaborator

No description provided.

if lang in messages.item_messages
else messages.item_messages[Language.ENGLISH]
),
centered=messages.centered,
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.

why isnt this just messages.centered?

Also pretty sure this should be messages.get("Centered", False)

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.

this hasn't been addressed yet fwiw.

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.

  1. Because the positional argument there is max_width, which is excluded
  2. I accidentally set the default to false in the schema, which is fixed 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.

i was mixing stuff up for both, oops.

@Miepee Miepee merged commit 4c0debe into main Feb 13, 2025
8 checks passed
@biosp4rk biosp4rk deleted the centered-text branch December 13, 2025 02:07
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.

2 participants