-
- Downloads
Milestone 3A: Add fixed width behavior to dashboard filters (#38585)
* Migration adding 'width' to Dashboards 3 migrations: - 1st adding the width column with default value of 'fixed' - 2nd updating all existing dashboards to have width 'full', which corresponds to what the current behaviour is (will be the 'old' behaviour after the fixed-width project lands). - The rolloback here is necessary but we don't care what happens as the column will be dropped immediately in the next rollback anyway - 3rd sets the notNullableConstraint. DefaultNull is 'full' here, just in case there's an existing dashboard whose width value is not yet set from the 1st migration. Don't know how that could happen, but its here in case * Dashboard PUT api endpoint accepts width changes and updates appdb update-dashboard function now is aware of the :width key so those changes can end up in the transaction. Also added a width test that asserts that the value's default is "fixed", it can be changed, eg. to "full", but cannot be changed to other values. * Add width to revision tests * Fix dashboard revision tests. :width key is now needed in some revision tests. As well we need a string communicating that the :width setting has changed from 'full' to 'fixed' or vice-versa. * Fix comments/remarks in migration to be accurate * Attempt to fix default not working mysql/mariadb * Set default in dashboard model Signed-off-by:Adam James <adam.vermeer2@gmail.com> * Revert default :width 'fixed' value. * Explicitly add default value 'fixed' for MySQL/MariaDB * dashboard fixed width FE implementation * adjust popover shadow styling as the popover was blending in with the header * adjust extra button popover offset * add E2E to validate behavior * add fixed width container to public dashboards * fix public embedding not respecting dashboard width setting * add test for public dashboards * Fix embed test failures * add fixed width to x-ray dashboards * reduce code duplication * move FixedWidthContainer into DashboardGridConnected - We get the fixed-width w/o code duplication across AutomaticDashboardApp, PublicDashboard, and Dashboard * update E2E tests to reflect actual intended behavior * fix type errors * tweak default width of cards to accomodate fixed width * adjust tooltip button after merging changes from master * fix bar chart test failure * fix dashboard filters date test failure - new ellipsis button in dashboard edit mode broke the test * adjust test for new fixed width dashboards * adjust test for new fixed width dashboards * adjust test for new fixed width dashboards * bump funnel default width * adjust test for new fixed width dashboards * Fix double overlay for LineAreaBarChart dash-cards - No longer has an overlay that persists from editing mode to viewing mode - Adjust click-behavior tests to account for new grid-width. Old test relied on a dashcard being taller than they will now default to being. i.e. Move the chosen point/row to down an index. * Merge FixedWidthContainer with dashboard grid div * supress brush events while editing timeseries line charts on dashboard * fix failing test - test failed because I did not update this assertion based on previous changes * fix broken link input in editing mode * initial commit * fix pointer events issue on text cards in edit mode * update tests for new dash-card default sizes * update tabs card moving test because of new default card sizes * Include apply filters button in fixed width behavior * tweak - do not need centering if full width * fix apply filters button * add fixed width behavior to dashboard filters in edit mode * remove double border in edit mode * evenly space parameters/filters in edit mode * update test to account for moving location of edit mode filters * Milestone 3B: Fix dashboard tabs and filters order on a public dashboard page (#38574) * Add `className` prop to `DashboardTabs` * Fix tab panel position in public dashboards * adjust filters container padding * Address feedback --------- Co-authored-by:
Jesse Devaney <22608765+JesseSDevaney@users.noreply.github.com> --------- Signed-off-by:
Adam James <adam.vermeer2@gmail.com> Co-authored-by:
Adam James <adam.vermeer2@gmail.com> Co-authored-by:
adam-james <21064735+adam-james-v@users.noreply.github.com> Co-authored-by:
Kyle Doherty <5248953+kdoh@users.noreply.github.com> Co-authored-by:
Aleksandr Lesnenko <alxnddr@gmail.com> Co-authored-by:
Anton Kulyk <kuliks.anton@gmail.com>
Showing
- e2e/test/scenarios/dashboard-filters/reproductions/17775-filter-custom-column-date.cy.spec.js 3 additions, 1 deletion.../reproductions/17775-filter-custom-column-date.cy.spec.js
- frontend/src/metabase/dashboard/components/Dashboard/Dashboard.styled.tsx 21 additions, 14 deletions...abase/dashboard/components/Dashboard/Dashboard.styled.tsx
- frontend/src/metabase/dashboard/components/Dashboard/Dashboard.tsx 22 additions, 14 deletions...src/metabase/dashboard/components/Dashboard/Dashboard.tsx
- frontend/src/metabase/dashboard/components/DashboardGrid.styled.tsx 3 additions, 4 deletions...rc/metabase/dashboard/components/DashboardGrid.styled.tsx
- frontend/src/metabase/dashboard/components/DashboardGrid.tsx 3 additions, 3 deletionsfrontend/src/metabase/dashboard/components/DashboardGrid.tsx
- frontend/src/metabase/dashboard/components/DashboardTabs/DashboardTabs.tsx 3 additions, 1 deletion...base/dashboard/components/DashboardTabs/DashboardTabs.tsx
- frontend/src/metabase/dashboard/containers/AutomaticDashboardApp.jsx 14 additions, 8 deletions...c/metabase/dashboard/containers/AutomaticDashboardApp.jsx
- frontend/src/metabase/parameters/components/ParameterWidget/ParameterWidget.styled.ts 1 addition, 2 deletions...ters/components/ParameterWidget/ParameterWidget.styled.ts
- frontend/src/metabase/public/components/EmbedFrame/EmbedFrame.styled.tsx 16 additions, 3 deletions...tabase/public/components/EmbedFrame/EmbedFrame.styled.tsx
- frontend/src/metabase/public/components/EmbedFrame/EmbedFrame.tsx 36 additions, 25 deletions.../src/metabase/public/components/EmbedFrame/EmbedFrame.tsx
- frontend/src/metabase/public/containers/PublicDashboard.jsx 4 additions, 6 deletionsfrontend/src/metabase/public/containers/PublicDashboard.jsx
- frontend/src/metabase/public/containers/PublicDashboard.styled.tsx 15 additions, 5 deletions...src/metabase/public/containers/PublicDashboard.styled.tsx
Loading
Please register or sign in to comment