Fix broken refresh button in ServerBrowser

[surfswift213us]

Bug Description

The refresh button in the ServerBrowser class is not working correctly after recent code updates. When the button is pressed, it doesn't properly refresh the server list as it did in the previous version.

What Changed

Comparing the old and new code, several critical changes were made that affected the refresh functionality:

1. Missing Button State Reset: In the new code, we fail to reset is_refreshing to false and refresh_button.disabled to false after refreshing. The previous code had these lines in _on_refresh_pressed():

is_refreshing = true
refresh_button.disabled = true
await _refresh_server_list()
is_refreshing = false  # This line is missing in new code
refresh_button.disabled = false  # This line is missing in new code

2. Removed Authentication Logic: The authentication check before refreshing was removed. The old code had:

# Re-authenticate if needed
if dns_manager.has_method("auth_and_sync_if_needed") and (dns_manager.session == null or dns_manager.session.is_expired()):
    var reok = await dns_manager.auth_and_sync_if_needed()
    if not reok:
        push_error("Authentication failed. Press Host to log in.")
        return

3. Removed Initial and Auto-Refresh Timers: The new code removed several auto-refresh related components:
   • Initial refresh timer setup and logic
   • The _try_refresh_with_auth() method
   • The auto-refresh state management methods

Steps to Fix

1. Restore the button state reset after refresh:

func _on_refresh_pressed() -> void:
    if is_refreshing:
        return
        
    # Re-authenticate if needed
    if dns_manager.has_method("auth_and_sync_if_needed") and (dns_manager.session == null or dns_manager.session.is_expired()):
        var reok = await dns_manager.auth_and_sync_if_needed()
        if not reok:
            push_error("Authentication failed. Press Host to log in.")
            return

    is_refreshing = true
    refresh_button.disabled = true
    await _refresh_server_list()
    is_refreshing = false  # Add this line back
    refresh_button.disabled = false  # Add this line back

2. Consider restoring the authentication logic in a separate method to ensure proper session handling before attempting to fetch server data.

Expected Behavior

After the fix, the refresh button should:

1. Disable itself while refreshing to prevent multiple clicks
2. Re-enable itself after refresh is complete
3. Properly update the server list with fresh data

Additional Notes

Other features that were removed and may be worth restoring:

• Auto-refresh functionality with configurable interval
• Initial refresh after a short delay
• Proper authentication and session management

These were likely important features that improved the user experience by keeping the server list up-to-date without manual refresh.