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

[EventGhost] - Enhancement - ConfigPanel.py, Equalizes the widths of the StaticText controls when using AddLine #287

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kdschlosser
Copy link
Member

When using the AddLine method of eg.ConfigPanel this causes the labels and controls to be rendered in a not so pleasing manner. I have added the equalizing of even numbered columns if the control is a wx.StaticText instance and if there are more column items in that specific row. it will add these instances into matching groups based on column number. and each grouping of controls then get passed to EqualizeWidths.

This produces a much nicer looking dialog. without messing up any description lines or help lines.

I am unsure if there are any use cases where this may cause an issue. I have not been able to locate one So be sure to test it if you know of a plugin that uses AddGrid or AddLine in an odd manner.

I could target this to be specific to AddLine. I would have to create the Controls if any text is present in the AddLine method instead of where they are currently created in the AddGrid method.

When using the AddLine method of eg.ConfigPanel this causes the labels and controls to be rendered in a not so pleasing manner. I have added the equalizing of even numbered columns if the control is a wx.StaticText instance and if there are more column items in that specific row. it will add these instances into matching groupes based on column number. and each grouping of controls then get passed to EqualizeWidths.

This produces a much nicer looking dialog.
@kdschlosser kdschlosser added this to the v0.5.1 milestone Nov 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants