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

Correct the Properties tab in showcase.py #650

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

Conversation

ZidYaBouzid
Copy link

In the Properties tab of showcase:

1- FontScale slider setup value as positinal argument was not taken into account (got lost in the new functions for v1.0 I guess). I made it a keyword argument and the slider is correctly positionned now.

2- Same for the TransparencyScale slider that was positioned at 0 instaed of 100.

3- There is a mismatch between inputs widgets and their labels, respectively in the right and left panedFrames. For a reason I ignore, the FontScale slider was displayed at the top of the right paned frame, while the coresponding label was at the bottom of the left panedFrame. This may have something to do with issue #301 "Not happy with layout" but I just tried appJar, created a GitHub account at last, and my time on this is up !

In the Properties tab of showcase:

1- FontScale slider setup value as positinal argument was not taken into account (got lost in the new functions for v1.0 I guess). I made it a keyword argument and the slider is correctly positionned now.

2- Same for the TransparencyScale slider that was positioned at 0 instaed of 100.

3- There is a mismatch between inputs widgets and their labels, respectively in the right and left panedFrames.
For a reason I ignore, the FontScale slider was displayed at the top of the right paned frame, while the coresponding label was at the bottom of the left panedFrame. This may have something to do with issue jarvisteach#301 "Not happy with layout" but I just tried appJar, created a GitHub account at last, and my time on this is up !
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.

None yet

1 participant