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

buttons in turtledemo disappear on small screens #62332

Closed
JanKanis mannequin opened this issue Jun 4, 2013 · 7 comments
Closed

buttons in turtledemo disappear on small screens #62332

JanKanis mannequin opened this issue Jun 4, 2013 · 7 comments
Labels
type-bug An unexpected behavior, bug, or error

Comments

@JanKanis
Copy link
Mannequin

JanKanis mannequin commented Jun 4, 2013

BPO 18132
Nosy @terryjreedy
Files
  • 7cdaf0223e67.diff
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2014-07-23.19:04:42.652>
    created_at = <Date 2013-06-04.13:46:45.135>
    labels = ['type-bug']
    title = 'buttons in turtledemo disappear on small screens'
    updated_at = <Date 2014-07-23.19:04:42.650>
    user = 'https://bugs.python.org/JanKanis'

    bugs.python.org fields:

    activity = <Date 2014-07-23.19:04:42.650>
    actor = 'terry.reedy'
    assignee = 'none'
    closed = True
    closed_date = <Date 2014-07-23.19:04:42.652>
    closer = 'terry.reedy'
    components = ['Demos and Tools']
    creation = <Date 2013-06-04.13:46:45.135>
    creator = 'JanKanis'
    dependencies = []
    files = ['30469']
    hgrepos = ['195']
    issue_num = 18132
    keywords = ['patch']
    message_count = 7.0
    messages = ['190596', '190652', '219295', '219490', '219713', '219719', '223762']
    nosy_count = 2.0
    nosy_names = ['terry.reedy', 'JanKanis']
    pr_nums = []
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'behavior'
    url = 'https://bugs.python.org/issue18132'
    versions = ['Python 2.7', 'Python 3.4', 'Python 3.5']

    @JanKanis
    Copy link
    Mannequin Author

    JanKanis mannequin commented Jun 4, 2013

    If the window of the turtledemo is small enough, the start/stop/clear buttons disappear. This is specifically a problem when running on a netbook with a small screen, because the buttons are never shown. For a newcommer checking out the demo app this could be very confusing as there is no indication on what went wrong, and the notification area just says 'press start button'.

    @JanKanis JanKanis mannequin added the type-bug An unexpected behavior, bug, or error label Jun 4, 2013
    @JanKanis
    Copy link
    Mannequin Author

    JanKanis mannequin commented Jun 5, 2013

    I've created a patch, replacing the packed layout by a gridded layout

    @terryjreedy
    Copy link
    Member

    On my widescreen, the buttons are completely gone when I shrink vertically to about 720 pixels. So they must be gone on a 600x800 screen or 720x1280 small widescreen and perhaps on a non-fullscreen window on a 780x10xx screen. I agree with fixing this if possible.

    The patch applies cleanly to 3.4 and fixes the problem with the buttons. However, it introduces a problem with the vertical canvas scrollbar, on the right. It does not appear when it should and does not scroll the entire canvas when it does appear. Try it with bytedesign, and see if you can fix it with an altered patch.

    @terryjreedy
    Copy link
    Member

    I believe I have read more than one warning to not mix grid and pack in the same master, as your patch does. For example: http://effbot.org/tkinterbook/grid.htm, An option would be to grid everything. I believe grid would also fix issue bpo-21597, so I may try it and see if it fixes both issue without introducing a scrollbar problem.

    @JanKanis
    Copy link
    Mannequin Author

    JanKanis mannequin commented Jun 3, 2014

    I wasn't aware that mixing grid and pack was a bad idea, as I was looking over turtledemo to learn how to use tkinter. The patch replaces a packing in one frame with a grid. All direct children of the graph_frame are gridded. graph_frame itself is still packed and so are the children of btn_frame which is a child of graph_frame.

    Also I wasn't able to find a clear definition of what is meant by 'master window' in the grid/pack warning, but supposedly that is the top level Tk() rather than a parent widget. I would offer to create a better patch but Lita Cho already did so in bpo-21597.

    @terryjreedy
    Copy link
    Member

    I had forgotten until I reread Lundh's note. I made the same assumption about 'master window'. Certainly 'conservative' is safest. Lita has not posted a patch, but since she is a (paid) summer intern (until mid August), I expect she will. If not, you can help finish this off before the next release cycle, which should be at least 3 months away. If you notice that nothing has happened by mid-August, please ping either or both issues.

    @terryjreedy
    Copy link
    Member

    Fixed by
    New changeset 60301b9982b2 by Terry Jan Reedy in branch '2.7':
    Issue bpo-21597: Turtledemo text pane can now be widened to view or copy complete
    http://hg.python.org/cpython/rev/60301b9982b2

    New changeset 0fb515063324 by Terry Jan Reedy in branch '3.4':
    Issue bpo-21597: Turtledemo text pane can now be widened to view or copy complete
    http://hg.python.org/cpython/rev/0fb515063324

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant