Covers all three scenarios listed in the original issue.

* Enable repro test

* Fix personal subcollection in item picker hidden

* added check for borderless buttons when setting compact padding

* fix my own silly regression

* added visual tests for run button regressions

* added minWidth to percy snapshot

* shoudl have read the docs

* Sync VIEWPORT_WIDTH and percy snapshot width

* grammar and microcopy tweaks

* update cypress test

* fix another test

* update test again

* Fix test's describe block

* Mention #16845 repro

* Assert native query text is displayed in Audit

* Enable the repro test

* Fix ad-hoc native query not displayed in Audit correctly

* Don't display native query sidebar in read-only mode

* Add a prop to bubble mouse events on SandboxedPortal

* Fix drag-n-drop on dashoboard's viz settings modal

* Enable repro test

* Handle composite PK filters as well as filtering by non unique PK fields

* unskip repro

* fix

* fix

* fixes

Previous implementation was dog slow. This one is faster

* add PK icon

* key -> label

* update api doc gen code

* update test

Fetch fields before parsing parameter values so we correctly parse values for boolean fields (#17999)

* add template tag fields as dependent metadata for native queries

* check for a field_id property added by getValueAndFieldIdPopulatedParametersFromCard

* Add NativeQuery dependentMetadata unit test

* add test for loadMetadataForQuery

* subscriptions and alerts audit

* Add tables

* Add tables

* Extract to lib

* Extract to lib

* Add modal

* Add modal

* Add modal

* Fix modals

* Fix modals

* Add children for displaying modals

* Add children for displaying modals

* Add children for displaying modals

* Add children for displaying modals

* Add edit dialogs

* Add edit dialogs

* Add edit dialogs

* Add edit dialogs

* Remove deletion

* Remove deletion

* Add users

* Add users

* Fix selection issues

* Add placeholder

* Add placeholder

* Add delete button

* Fix wrapping issues

* F

* Small fixes

* Add reloading

* fixes

* cypress specs

* fixes

* post rebase fixes

* rebase fixes

* review fixes

Co-authored-by: Alexander Polyankin <alexander.polyankin@metabase.com>

* restore original file name

* links

* show meaningful error for waterfall chart of multiple series questions

* increase readability

* Formatting

return a format string so it can be used over multiple rows. The rules
for formatting are quite annoying because there are defaults that are
assumed rather than present. IE, if you set a column as currency it
doesn't set the number style sometimes and defaults to USD without that
being in the settings. In the future would be far preferable to always
create a fully fleshed map of defaults rather than inferring them in
random places.

* appease our doc string supervisor

* Include column information for formatting numbers

really important for requiring two decimal places by default on doubles
and none on integers. Its close to correct ignoring it and using
"#,###.##" and this leaves non decimal places on integers and allows
only up to two on doubles, but things like 31.10 will format as 31.1 and
that is improper

* Remove unneeded metabase.test

* Update tests with nil column

* Get global viz settings in middleware, pass entire to formatting

Passing the entire viz settings to the formatting stuff makes the
calling code much simpler and the places where we merge global settings,
possibly get currency settings, etc, just in one place. We already had
the col information there so just use it.

* Remove public settings now that we get it from the metadata

* Check for existing result set when displaying
"Showing [X] rows" instead of the max limit

* added cypress test

* broke up nested ternary operators in QuestionRowCount

Fix #16810: Dashboard "isDirty" property is set to true after the Dashboard title is updated. (#17955)

* added isDirty flag to SET_DASHBOARD_ATTRIBUTES action

* added reducer tests for SET_DASHBOARD_ATTRIBUTES with isDirty

* lint