ESPConnect | Easy WiFi Manager with Captive Portal

This is what you currently have in your code:

you set self.espconnecting = False

you call _connect_esp()

which calls _show_conn_progress()

which sets self.espconnecting = True and starts the progressbar self.sp_pbar.start()

and then calls _update_conn_progress()

which checks the value of self.espconnecting. If self.espconnecting is True(which it currently is) connection continues and progress bar keeps rolling as expected. If self.espconnecting is False progress bar is stopped self.sp_pbar.stop()

Before .after() can make it's callback in 500ms, Control is passed back to _connect_esp which sets self.espconnecting = False. Then .after() calls _update_conn_progress() which is meant to keep the bar rolling,

but(Here is your problem): what is the last value of self.espconnecting? =False hence, control branches to self.sp_pbar.stop(), which stops the progrss bar. This is why when you comment that line out your code works as expected, because even if control branches there, there will be nothing to prevent the progress bar from working.

SOLUTION

Do not set self.espconnecting = False in _connect_esp() because before .after() makes it's callback in 500ms, control would have been passed back to _connect_esp() which sets self.espconnecting = False which prevents your progress bar from working.

This means you have to find another means to 'end the connection', once it gets started.

N.B: I really don't see the need for time.sleep(5) in the code.

Here is a possible way to go about the solving it: