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

Functioning Tix.Grid #43121

Closed
tzot mannequin opened this issue Mar 31, 2006 · 6 comments
Closed

Functioning Tix.Grid #43121

tzot mannequin opened this issue Mar 31, 2006 · 6 comments
Assignees

Comments

@tzot
Copy link
Mannequin

tzot mannequin commented Mar 31, 2006

BPO 1462222
Nosy @loewis
Files
  • Tix.diff: removed empty functions
  • 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 = 'https://github.com/loewis'
    closed_at = <Date 2006-04-10.08:35:37.000>
    created_at = <Date 2006-03-31.16:28:04.000>
    labels = ['expert-tkinter']
    title = 'Functioning Tix.Grid'
    updated_at = <Date 2006-04-10.08:35:37.000>
    user = 'https://bugs.python.org/tzot'

    bugs.python.org fields:

    activity = <Date 2006-04-10.08:35:37.000>
    actor = 'loewis'
    assignee = 'loewis'
    closed = True
    closed_date = None
    closer = None
    components = ['Tkinter']
    creation = <Date 2006-03-31.16:28:04.000>
    creator = 'tzot'
    dependencies = []
    files = ['7111']
    hgrepos = []
    issue_num = 1462222
    keywords = ['patch']
    message_count = 6.0
    messages = ['49887', '49888', '49889', '49890', '49891', '49892']
    nosy_count = 2.0
    nosy_names = ['loewis', 'tzot']
    pr_nums = []
    priority = 'normal'
    resolution = 'accepted'
    stage = None
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue1462222'
    versions = ['Python 2.5']

    @tzot
    Copy link
    Mannequin Author

    tzot mannequin commented Mar 31, 2006

    This patch to Tix.py provides the basic functionality
    needed for the tixGrid widget to work in Tix
    applications. As I unravel the Tix C / Tcl/Tk code and
    find out what more functionality is available, I will
    update the patch.

    There will be an associated patch providing a module to
    be used with tixwidgets.py as a demo.

    @tzot tzot mannequin closed this as completed Mar 31, 2006
    @tzot tzot mannequin assigned loewis Mar 31, 2006
    @tzot tzot mannequin added the topic-tkinter label Mar 31, 2006
    @tzot tzot mannequin closed this as completed Mar 31, 2006
    @tzot tzot mannequin assigned loewis Mar 31, 2006
    @tzot tzot mannequin added the topic-tkinter label Mar 31, 2006
    @tzot
    Copy link
    Mannequin Author

    tzot mannequin commented Mar 31, 2006

    Logged In: YES
    user_id=539787

    Here follows code (testgrid.py) I use to test the patched
    Tix.py, used as 'python -i testgrid.py'

    ###
    import Tix as tk
    from pprint import pprint
    
    r= tk.Tk()
    r.title("test")
    
    l=tk.Label(r, name="a_label")
    l.pack()
    
    class MyGrid(tk.Grid):
        def __init__(self, *args, **kwargs):
            kwargs['editnotify']= self.editnotify
            tk.Grid.__init__(self, *args, **kwargs)
        def editnotify(self, x, y):
            return True
    
    g = MyGrid(r, name="a_grid", selectunit="cell")
    g.pack(fill=tk.BOTH)
    for x in xrange(5):
        for y in xrange(5):
            g.set(x,y,text=str((x,y)))

    @tzot
    Copy link
    Mannequin Author

    tzot mannequin commented Apr 3, 2006

    Logged In: YES
    user_id=539787

    Accepting this patch closes bug 1036406.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Apr 10, 2006

    Logged In: YES
    user_id=21627

    That patch can't be right: it introduces empty function
    selection_adjust etc. These functions should, of course,
    invoke the underlying widget commands.

    Please provide a patch that can be used as-is. It might be
    incomplete (i.e. not expose all functionality), but for the
    functionality it provides, it should be correct.

    @tzot
    Copy link
    Mannequin Author

    tzot mannequin commented Apr 10, 2006

    Logged In: YES
    user_id=539787

    You are right. Changed empty functions to comments similar
    to those pre-existing in order to mark missing
    functionality. New patch provided.

    @loewis
    Copy link
    Mannequin

    loewis mannequin commented Apr 10, 2006

    Logged In: YES
    user_id=21627

    Thanks for the patch. Applied as r45229.

    @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
    Projects
    None yet
    Development

    No branches or pull requests

    0 participants