Skip to content

Option to rotate text (and only text) by 180°#136

Open
efogdev wants to merge 3 commits intogregdavill:mainfrom
efogdev:feat/rotate-text-180
Open

Option to rotate text (and only text) by 180°#136
efogdev wants to merge 3 commits intogregdavill:mainfrom
efogdev:feat/rotate-text-180

Conversation

@efogdev
Copy link
Copy Markdown

@efogdev efogdev commented Mar 6, 2025

Why? Because before:

Screenshot From 2025-03-05 04-03-12

After:

image

Made this for myself, idk if you actually need it :)

@arturo182
Copy link
Copy Markdown
Collaborator

It looks like you modified dialog_text_base.py by hand, that file is generated by wxFormBuilder from text_dialog.fbp. You should make your dialog changes using the builder and then generate a new .py file using it.

Also, I do wonder if instead of rotate, we should maybe call it flip, just in case in the future we want add a more granular rotation.

@efogdev
Copy link
Copy Markdown
Author

efogdev commented Mar 7, 2025

Oh. Thanks for clarification, never worked with wxWidgets

@efogdev
Copy link
Copy Markdown
Author

efogdev commented Mar 8, 2025

Apparently I need the exact version of wxFormBuilder, should I go for it? Or can I edit the fbp file directly?

@CrackXT
Copy link
Copy Markdown

CrackXT commented May 27, 2025

Hello efogdev,

don't you achieve the same effect with the Padding option?

I tried to understand the advantage of your option and couldn't see it for myself. Padding is “faster” with the same end result.

Kind regards
CrackXT

@efogdev
Copy link
Copy Markdown
Author

efogdev commented May 28, 2025

@CrackXT yep it can be achieved with padding, but when you copy&paste and rearrange/rotate labels a lot, it's easier to click a checkbox when needed. If you have a label with padding, and you need to rotate the text, you have to change the starting cap, the ending cap, and the 2 padding values.

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.

3 participants