Follow up after #34209

Co-authored-by: bryan <bryan.maass@gmail.com>

When a card or dashboard is pinned, the entity itself is not changing, so let's not send the `:event/card-update` or
`:event/dashboard-update` events in these cases.

* empty commit to allow draft pr

* dismiss undo notifications when saving a dashboard (#35243)

* dismiss undo notifications when saving a dashboard

* e2e test

* Update e2e/test/scenarios/dashboard/tabs.cy.spec.js

Co-authored-by: Oisin Coveney <oisin@metabase.com>

---------

Co-authored-by: Oisin Coveney <oisin@metabase.com>

* fix z-index issues

* Update frontend/src/metabase/dashboard/components/DashCard/DashCard.styled.tsx

Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>

* renames as per feedback on PR

* left align dashcard actions on small cards (#35327)

* update emotion to support container queries

* left align actions with container query

* use rems

* fix restore z index between resize handle and dashcard actions

* refactor: remove renderDashCardActions in favor of a component (#35366)

* move DashboardCardActionsPanel inside DashCardActionButtons

* move isDashCardTabMenuOpen state to DashCardACtionsButtons

* refactor: remove renderDashCardActions

* refactor: renamed to DashCardActionsPanel

* refactor import type MouseEvent

* refactor DashCardActionButtons folder -> DashCardActionsPanel

* refactor: move DashCardTabMenu to DashCardActionsPanel

* refactor: move buttons to their own subfolder so that it's clear they're sub-components of the actions panel

* feat: use name of card in undo toast message (#35378)

* feat: use name of card in undo toast message

* feat/refactor: truncate via css

* denis' patch to use Ellipsified

* make toast don't take up all space + add max width

* Update frontend/src/metabase/containers/UndoListing.styled.tsx

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

---------

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

---------

Co-authored-by: Oisin Coveney <oisin@metabase.com>
Co-authored-by: Kamil Mielnik <kamil@kamilmielnik.com>
Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>

* Fix newly created E2E CSV flakes

* Do not use `.within()` on multiple elements

* Always isolate the correct status element before assertions

`load-metabase!` now has an exclamation mark in the name
Fixes a breaking change in master caused by merging #35539 before #35530.

Cards are not touched, just DashboardCards and DashboardTabs are synchronized with loaded data

* Add a repro for the basic case

* Add more assertions

* Remove external tag

* Improve test name

* Add a test case for visited questions

* Fix initial state for lhc display name for visited questions

* Refactor duplication

* Update test names

* Skip test that still fails

* Revert "Fix initial state for lhc display name for visited questions"

This reverts commit 36aca5469dff6e711725e930eb1b29b6bef7dbef.

* Use getNotebookStep

* Skip test until issue is fixed

* Refactor to prepare for new requirement

* Hide personal collections and items when configuring click behavior

* Remove default export

* Add tests

* Fix type error

* Review: Simplify test util

* Update frontend/src/metabase-lib/parameters/utils/click-behavior.ts

Co-authored-by: Denis Berezin <denis.berezin@metabase.com>

* Consolidate collection util functions

* Reuse an existing function

* Fix conflict from rebasing off master

---------

Co-authored-by: Denis Berezin <denis.berezin@metabase.com>

* Display error instead of auto-fixing non-exiting tab id

* Make deleted target dashboard tab an invalid click behavior state

* Fix saving click behavior

* Add a test for showing error when the tab has been removed

* Add auto-fix for deleted tab when we don't render tabs dropdown

* Update test names

* Do not show new question link when database does not support nested queries

* Do not show new question link when nested queries are disabled

* Add a unit test for disabled nested queries

* Add a unit test for databases without nested queries support

keys in serialization are be sorted by historical order first and alphabetically if they are not mentioned

Co-authored-by: Bryan Maass <bryan.maass@gmail.com>

* Split visualizations E2E group

* Rename visualizations to `visualizations-charts`

* Update E2E matrix

* Attempting to fix x-ray back timeout issues with smaller card count

It looks like the flakes on https://www.deploysentinel.com/ci/analysis?tab=flake for "dashboard back navigation should display a back to the dashboard button in table x-ray dashboards" and "dashboard back navigation should display a back to the dashboard button in model x-ray dashboards" frequently fail at the `cy.wait("@dataset")` stage. I'm guessing this is because the dashboard code is so slow. This PR limits the number of cards produce to `MAX_CARDS` (5) in order to speed up card generation and help with the flakes.

* Adding a timeout to slow x-rays for back button. Hopefully, the small number of max cards makes this unnecessary, though.

* Bumped timeout to 15s instead of 5s. Looks like it was already 5. Again, hopefully the change to minimized the number of generated cards will fix this flake and not take as long.

* Fixing the `show` query param for x-rays

`show` in `core` can be either `:all` or an integer. However, the API layer only exposes the `:all` or `nil` options. This PR exposes the integer path as well.

Note that this is incremental to get flakes working faster. I'll still need to add API tests for this.

* Adding API unit tests to `metabase.api.automagic-dashboards-test` for `show` cases.

* Fixed deftest name to be unique.

* Fixing reference to new card in flaky test

Co-authored-by: Luiz Arakaki <luiz.arakaki@metabase.com>

Adding unit tests for bug bash. Closes #14775.

* [MLv2] Test for 27323

Fixes #27323

* Address PR review

Fixes #28609

* separate SMTP connection settings from other email settings

* update tests based on now setup flow

- some tests were removed as the behaviors of the app have changed
- some tests were updated to validate the new SMTP connection card and setup process

* improve tests

* replace styled component with mantine flex

* fix type errors

* refactor breadcrumbs to use native solution in SettingsBatchForm

* only show breadcrumbs if email is already configured

* catch and handle error if form submission fails

* fix tests

* remove dead code

* fix test failure and consolidate similar E2E tests

* update unit tests to account for new behavior

* fix type error - remove unused dispatch prop

* update unit tests to account for redirect behavior

* move dispatch to useEffect and convert file to typescript

* fix button state on save error

* add unit tests for hiding SMTP configuration when cloud-hosted

* enable actions on databases on a per-test basis

* debug logging

* try some stuff

* optimistic sync

* put some stuff back

* revert formatting changes

* revert debug changes [ci skip]

* Adding Stacked Checkbox component

* Switching to checkbox variant

* PR Feedback

* Storybook cleanup

* including description