How to make a tkinter button open only one Toplevel?

Question

I need a tkinter button to open only one Toplevel but I tried using a counter that changes to 2 when one Toplevel is made and a if loop so it checks if the counter is 1, (it will make a Toplevel window if its 1)

but when I run the program I can make many windows by clicking the button multiple times

I think the solution of using a counter doesn't work

    def menu_window(self): # this is in a class

    self.frame4 = tk.Frame(self.master, padx=30, pady=30)
    self.frame4.grid()

    Counter = 1  ### COUNTER 
    button2 = tk.Button(self.frame4, text="Review your Quiz", command=lambda: PreviewQuiz(self.master, self.frame4,
                                                                                          Counter))
    button2.grid(row=3, column=2, padx=40, pady=15, ipadx=28)
    button3 = tk.Button(self.frame4, text=" Start your Quiz ")
    button3.grid(row=4, column=2, padx=40, pady=5, ipadx=30)


class PreviewQuiz:
    def __init__(self, master, frame4, Counter):
        if Counter == 1:   # CHECK IF COUNTER IS 1
            self.master = master
            self.review_q = tk.Toplevel(self.master)
            self.frame5 = tk.Frame(self.master, padx=50, pady=20)
            self.frame5.grid()
            self.Counter = 2  # SET COUNTER TO 2

Can you please fix the indentation, and gives us minimalistic working example. Also as @figbeam said you need to change `Counter = 1.` to `Counter = 1` otherwise it will be a `float` not an `int` — TheLizzard, Mar 04 '21 at 10:08

score 2 · Accepted Answer · answered Mar 04 '21 at 14:59

Just disable the button when the user clicks on it and enable it only when the top-level is closed.

Here is an example taken from the code you provided in your newest post:

import tkinter as tk

class Run:
    def __init__(self, master):

        self.master = master
        
        self.button = tk.Button(master, text="TopLevel", command=self.make_new)
        self.button.pack()

    def make_new(self):
        self.button['state'] = 'disabled'

        new = tk.Toplevel(self.master)

        lbl = tk.Label(new, text='only one topLevel')
        lbl.pack()

        new.protocol("WM_DELETE_WINDOW", lambda : self.button.configure(state='normal') or new.destroy()) # or make a method to change the state
        

master1 = tk.Tk()
i = Run(master1)
master1.mainloop()

score 1 · Answer 2 · answered Mar 04 '21 at 10:00

1

Your code does not run, which makes it difficult to debug. There are however a few things:

When you assingn a value to counter in menu_window() you end the value with a ".", which makes it a float, not an integer.

You call PreviewQuiz with the counter argument, but when you update the counter you do it to an instance variable of PreviewQuiz, not of menu_window() class so next time you call PreviewQuiz you will still use 1.0.

answered Mar 04 '21 at 10:00

figbeam

7,001
2
12
18

where should I update the counter variable? – coder_not_found Mar 04 '21 at 10:02
1

You can make it an instance variable of the class containing the `menu_window()` method. It will still be available to `PreviewQuiz` as `self.master.Counter`. – figbeam Mar 04 '21 at 10:15
Or are you tryint to implement the singleton pattern? In that case, have a look at [Creating a singleton in Python](https://stackoverflow.com/questions/6760685/creating-a-singleton-in-python) and [the Borg design pattern](https://github.com/ActiveState/code/tree/73b09edc1b9850c557a79296655f140ce5e853db/recipes/Python/66531_SingletWe_dont_need_no_stinksingletBorg_design) – figbeam Mar 04 '21 at 10:18
if I'm updating the value of the counter variable in menu_window() ; setting counter to 2; should I use a if loop with winfo_exists function? – coder_not_found Mar 04 '21 at 12:37
also I think the link you provided is not related to this question . – coder_not_found Mar 04 '21 at 12:39

How to make a tkinter button open only one Toplevel?

2 Answers2

Linked

Related