resolves #35880

* Fix "slugify collection url" flake

* Remove the `@flaky` tag

* try to restore the original position even when another card stole it

* avoid calling fetchCardData in undoRemoveCardFromDashboard for virtual dashcards

* test: e2e test for undo movement and card removal

* test to make the ci pass

* empty commit

* restore old scrollbehaviour

* refactored the magic number in a function so that also docs are in a single place

* Fix recent-views flake(s)

* Remove the `@flaky` tag

* Visit model query directly

* Use helper to save model metadata changes

* Limit the number of results even more

* Improve assertions

* Increase the viewport width in order to see all columns

Spans on stats for massive models are very long, on the order of 4s.

The bad span seems to encompass a small amount of logging and assertion
logic, including a with-out-str pprint, which I've found to be
notoriously slow for large objects. This PR slims down the logged object.

If this is the culprit we should see a substantial performance improvement.

* Adding more spans to understand stats `publish-event!`

https://stats.metabase.com/dashboard/2117

 still takes 4s to load and
the current spans don't seem to provide fine granularity around the
hot spot, but it seems to be happening in the early stages of
`publish-event!`. These extra spans provide finer granularity.

* Adding what is probably a non-needed span, but we want to over-instrument this, if anything. Once we id the hotspot we can remove what we want.

* add more spans

* remove :object from new spans

---------

Co-authored-by: Noah Moss <noahbmoss@gmail.com>

* [E2E] Fix dashboard card resize flake

* Remove the `@flaky` tag

* Add test to the slow group

* [MLv2] Put duplicate implicit join columns in separate column-groups

* Add tests for cards without id fields in metadata

* Fix linter

* Fix test

* Add Mantine `Modal`

* Add guidelines link

* Fix heading in storybook

* Auto apply dashboard filters with the same field (#35280)

* Add explanatory toast and `Undo` when auto-applying filters (#35461)

* Add data test ID to dashboard sidebar

* Auto apply filters to added cards

* Discard changes to frontend/src/metabase/parameters/utils/dashboards.ts

* Discard changes to frontend/src/metabase/dashboard/components/Sidebar.jsx

* Make sure that util functions work

* Attempt to split code to where it belongs

* Remove useless function

* Remove useless function

* Discard changes to frontend/src/metabase/dashboard/utils.ts

* Change names to match nomenclature

* Move more files around

* More fixes to types

* Move to helpers

* Solve some of the top levels

* Attempt to fix types

* Attempt to make types work without exploding

* Add undo elements for mapping existing parameters and adding a new card with a parameter

* Add undo toast to tests

* First attempt at setting up undos and disabling behavior for cards that have had their auto-wiring undone

* Add undo toasts and post-undo toast

* More changes

* Discard changes to frontend/src/metabase-lib/parameters/utils/targets.ts

* Discard changes to frontend/src/metabase-types/api/card.ts

* Split into files

* Split files

* Discard changes to e2e/test/scenarios/permissions/admin-permissions.cy.spec.js

* Fix typescript errors

* Make unit tests work

* Remove toasty states

* Add type for more complex parameter

* Fix type errors

* Remove disabled cards logic

* Fix e2e tests

* Add e2e tests to make sure everything works

* Update e2e/test/scenarios/dashboard-filters/parameters.cy.spec.js

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

* Use helpers in tests and refactor to keep dashcardAttributes in dashboard code

* fix undo timeout leak by clearing timeouts when undo is dismissed

* e2e: add e2e for timeout edge case

* test: add unit test for undo

* fix typo

---------

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>
Co-authored-by: Nicolò Pretto <info@npretto.com>

* remove e2e test with cy.wait (#35850)

* removes a test that uses a cy.wait

* Revert "removes a test that uses a cy.wait"

This reverts commit 52d51140bbd3bc053fc7272d42a53feaa9c1c1bb.

* use cy.clock and cy.tick instead of cy.wait

---------

Co-authored-by: Mahatthana (Kelvin) Nomsawadi <me@bboykelvin.dev>
Co-authored-by: Nicolò Pretto <info@npretto.com>

when linkType is "url", serialization broke since there is no corresponding model

Resolves #35855

* use normal font weight for 'values'

* update label styles across inputs

Add test for #14648 'Pivot action on a scalar in an x-ray results in a broken visualization' (#35706)

* Adding additional instrumentation to `publish-event!`

The `publish-events!` defmethod has quite a few code branches and we're trying to figure out if one of these is causing the slowness described in #33499 as well as stats dashboard [2117](https://stats.metabase.com/dashboard/2117). This PR should provide additional information as to where the delays might be occurring.

It adds the following spans/changes:
- An `:events/initialized` attribute at the top level of `publish-event!`
- `record-view!` span
- `record-event!` span
- `update-users-recent-views!` span

If we merge this PR we can decide if these changes are worthwhile and revert if we don't like the additional layer of wrapping.

* Reinstating record-event! span to resolve rebase conflict.

* Fixing bad merge

The `values-from-card-query` constructs a new query to determine
enumerable values for filter drop-downs in dashboards. It does this
by generating a simple query from the passed in query with the
filtered value as a query filter. This query didn't know about
expressions which the filter might use. The fix was to just add
in the expressions from the base query.

Fixes #12985

also adds quickcheck tests for user-id and model-id precedence