Tkinter Button to creat entries

Your code stops as soon as the return statement is reached, and thus your loop never goes further than the first iteration.

On the other hand, if you remove the return statement, there is nothing in place for stopping your loop so it keeps running until it has created all your widgets.

I took an approach using a generator to solve your issue, such that instead of having a function new_entry(), instead you have a function generating the entries, and another function placing the entries when the generator is called.

I also simplified your code a little at the widget generation, and changed a spelling error (spent instead of spend). I also replaced from tkinter import * with import tkinter as tk, to not pollute your namespace.

import tkinter as tk

root = tk.Tk()
root.geometry("274x520+868+240")
root.minsize(120, 1)
root.maxsize(3284, 1068)
root.resizable(0, 0)
root.title("Learning")
root.configure(background="black")


def entry_generator():
    for i in range(30, 490, 30):
        yield place_entry(i)


def place_entry(i):
    e1 = tk.Entry(root)
    e2 = tk.Entry(root)
    e1.place(y=i, width=133)
    e2.place(x=141, y=i, width=137)
    print(e1, e2)


ne = entry_generator()
b1 = tk.Button(root, command=lambda: next(ne), text='+', background='darkgrey')
b1.place(y=499, width=276)

msg1 = tk.Label(root, text='Activity:', background='darkgrey')
msg1.place(width=133, height=29)

msg2 = tk.Label(root, text='Time Spent:', background='darkgrey')
msg2.place(x=141, width=137, height=29)

root.mainloop()

def new_entry():
    n = [n for n in range(30, 490, 30)]

    for i in n:
        e1 = Entry(root)
        e2 = Entry(root)
        e1.place(y=i, width=133)
        e2.place(x=141, y=i, width=137)

        print(e1, e2)
        return e1, e2 ```

The return is the problem. Look at following sample code:

def iterate():
    for i in range(10):
        print(i)
        return i
val = iterate()
print(val)

When you run this it will print:

0
0

Because, return makes the function stop immediately and return something. If you really had to return every value, you could create a list, append the things to it, and return that list after the loop:

def iterate():
    arr = []
    for i in range(10):
        print(i)
        arr.append(i)
    return arr
val = iterate()
print(val)

which will print:

0
1
2
...
10
[0, 1, 2, ... 10]

or use the yield keyword (it will return the value without making the loop stop):

```python
def iterate():
    for i in range(10):
        print(i)
        yield i
val = iterate()
print(val)
print([i for i in val])

will print:

0
1
2
...
10
<generator object iterate at 0x7fb64e584f68> 
[0, 1, 2, 3 ... 10]

With yield, val will be a generator object, over which you must iterate again to get the values out of it. But, as the return value isn't used anywhere in your code, you can just remove the return e1, e2.

Another method achieve might be to keep a count of the entry and multiply the count with the height of the Entry widget

Here is an example:

count = 0

def new_entry():
    global count
    e1 = Entry(root)
    e2 = Entry(root)

    count += 1
    y_pos = count*e1.winfo_reqheight()
    e1.place(x=0, y=y_pos*1.5, width=133) # change the multiplication factor to change the padding b/w entry
    e2.place(x=141, y=y_pos*1.5, width=137) # change the multiplication factor to change the padding b/w entry

    print(e1, e2)

Also note that it might be better to use grid layout for such programs.