Skip to content
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

Merged
merged 6 commits into from
Feb 20, 2023

Conversation

rotorman
Copy link
Member

Adds two new options to value widget to align label and value optionally also to center or right border of the widget zone:

grafik

grafik

I tried my best with translations, but am pretty sure that there are some languages that I did not manage to match perfectly.

@rotorman rotorman added enhancement ✨ New feature or request color Related generally to color LCD radios widget labels Jan 22, 2023
@rotorman rotorman force-pushed the valueWidgetAlignRightOpt branch from 183ed18 to 0d2458d Compare January 22, 2023 15:30
@Eldenroot
Copy link
Contributor

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!

@elecpower
Copy link
Collaborator

@rotorman I assume you would like me to sync Companion so got it

@rotorman
Copy link
Member Author

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!

Sure, what should I fix?

@elecpower yes, please! Thx!

@Eldenroot
Copy link
Contributor

I added a comment
image

@elecpower
Copy link
Collaborator

@rotorman done

@pfeerick
Copy link
Member

I added a comment

@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 ;)

@pfeerick pfeerick self-assigned this Jan 26, 2023
@pfeerick pfeerick added this to the 2.9 milestone Feb 5, 2023
Copy link
Member

@pfeerick pfeerick left a 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.

@pfeerick pfeerick added documentation Improvements or additions to documentation lua-api Lua API related labels Feb 20, 2023
@pfeerick pfeerick merged commit 97c57cf into EdgeTX:main Feb 20, 2023
@rotorman rotorman deleted the valueWidgetAlignRightOpt branch February 20, 2023 07:30
@phileville
Copy link
Collaborator

Would it be possible to also add these options to the "Text" widget as well?

@rotorman
Copy link
Member Author

rotorman commented May 9, 2023

@phileville Please see PR #3574

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
color Related generally to color LCD radios documentation Improvements or additions to documentation enhancement ✨ New feature or request lua-api Lua API related widget
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants