-
-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(ui): Value widget text alignment options #3080
Conversation
…right border of the widget zone.
183ed18
to
0d2458d
Compare
CZ is fine, just only one typo (it was here before your PR, if you can fix it in this PR would be nice). Thx! |
@rotorman I assume you would like me to sync Companion so got it |
Sure, what should I fix? @elecpower yes, please! Thx! |
@rotorman done |
@Eldenroot Just FYI - it says pending in that screenshot - so you didn't actually submit the comment/review... hence why it didn't show up ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Works fine when coming from a version of the widget without the extra options (i.e. no change), aligns nicely, and opts survived being read/written with companion.
Would it be possible to also add these options to the "Text" widget as well? |
@phileville Please see PR #3574 |
Adds two new options to value widget to align label and value optionally also to center or right border of the widget zone:
I tried my best with translations, but am pretty sure that there are some languages that I did not manage to match perfectly.