Suppress only the expected errors from the retried dialog click

[adamtheturtle]

Closes #1490.

_open_add_database_dialog clicks the add-database button, then clicks it a second time inside contextlib.suppress(WebDriverException) because the dialog sometimes does not open on the first click. WebDriverException is the base of the entire Selenium exception hierarchy, so that suppressed everything — a dead session, a browser crash, a chromedriver protocol error — and turned it into the 30 second timeout on the next wait, with no trace of the real cause.

It now suppresses only the three errors that mean the first click did open the dialog: the button is covered by the dialog (ElementClickInterceptedException), hidden (ElementNotInteractableException), or gone from the DOM (StaleElementReferenceException). Anything else propagates.

I checked which one actually fires, against the real target manager — click the button, then click the same element again:

first click ok
second click raised: selenium.common.exceptions.ElementClickInterceptedException
  message: element click intercepted: Element <button class="btn" id="add-dialog-btn" ...>
           is not clickable at point (1407, 186). Other elem...
dialog name field present: True

So the common path is ElementClickInterceptedException, which is in the suppressed set, and the dialog is open afterwards as expected. The other two are kept for the variants where the portal hides or re-renders the button instead of covering it.

WebDriverException is no longer imported.

🤖 Generated with Claude Code

https://claude.ai/code/session_01YJDy3qA1LyNTh11zS98ukM