Skip to content
Snippets Groups Projects
This project is mirrored from https://github.com/metabase/metabase. Pull mirroring updated .
  1. Oct 10, 2024
  2. Oct 08, 2024
  3. Oct 07, 2024
  4. Oct 04, 2024
  5. Oct 02, 2024
    • Emmad Usmani's avatar
      add sunburst to pie chart (#47208) · 61e610b3
      Emmad Usmani authored
      * add sunburst to pie chart
      
      * fix type errors
      
      * consider all slice percent values for formatting percent on chart
      
      * fix lint error
      
      * refactor nested functions outside of parent
      
      * create and use getArrayFromMapValues helper
      
      * fix bug with percent significant digits
      
      * improve color picker
      
      * fix total visibility in small dashcards
      
      * fix missing field remappings
      
      * use aggregated value for click behavior
      
      * update minimum slice percentage logic
      
      * fix total text truncation
      
      * remove colors from tooltip for middle and outer slices
      
      * update e2e spec
      
      * update snapshot
      
      * update ring setting titles
      
      * address feedback
      
      * update comment
      Unverified
      61e610b3
  6. Sep 13, 2024
  7. Sep 04, 2024
  8. Aug 30, 2024
  9. Aug 29, 2024
    • Emmad Usmani's avatar
      allow re-ordering and re-naming pie chart slices (#46607) · 0139fd78
      Emmad Usmani authored
      * allow re-ordering and re-naming pie chart slices
      
      * aggregate rows with same dimension value
      
      * show warning when dimension has unaggregated values
      
      * change colors from sidebar
      
      * implement renaming
      
      * styles for slice name widget
      
      * handle numeric slice keys for binned values relative dates etc
      
      * implement slice disabling
      
      * remove old colors setting
      
      * implement merging new and removing existing rows from query results
      
      * fix formatting bugs
      
      * port settings to static viz
      
      * fix bug with unaggregated data
      
      * address todos
      
      * add loki test
      
      * fix measure column settings widget erroring
      
      * fix settings sidebar crashing
      
      * add e2e tests
      
      * update snapshots
      
      * fix pie_chart.cy.spec.js CI-only flakes
      
      * fix #21504 repro failure
      
      * fix click action value
      
      * use nestedSettings instead of seriesSettings
      
      * fix settings sidebar crashing
      
      * fix row with 0 key not being draggable
      
      * fix flaky test
      
      * filter other slices from sidebar
      Unverified
      0139fd78
  10. Aug 27, 2024
  11. Aug 22, 2024
  12. Aug 21, 2024
  13. Aug 19, 2024
  14. Aug 16, 2024
    • Aleksandr Lesnenko's avatar
      use echarts tooltip (#46359) · c6141aa7
      Aleksandr Lesnenko authored
      
      * use ECharts tooltip on cartesian charts
      
      * fix tooltip lag when changing hovered series
      
      * facelift tooltip
      
      * fix static viz crashes due to tooltip option
      
      * facelift
      
      * pie chart echarts tooltip
      
      * facelift
      
      * facelift
      
      * facelift
      
      * tweaks
      
      * e2e specs
      
      * return back the pie other slices tooltip
      
      * more specs
      
      * specs
      
      * fix pie total
      
      * fix tooltip overflow
      
      * fix specs
      
      * fix infinite rerender in embedding mode for charts that have warnings
      
      * Update Loki Snapshots
      
      ---------
      
      Co-authored-by: default avatarMetabase Automation <github-automation@metabase.com>
      Unverified
      c6141aa7
  15. Aug 14, 2024
  16. Aug 13, 2024
  17. Aug 12, 2024
  18. Aug 08, 2024
    • Emmad Usmani's avatar
      automatically use compact formatting for pie chart total (#46505) · a4bec904
      Emmad Usmani authored
      * automatically use compact formatting for pie chart total
      
      * add story and snapshots
      Unverified
      a4bec904
    • Mahatthana (Kelvin) Nomsawadi's avatar
      Follow-up milestone 2 for [Epic] Re-design transparent theme for static... · a4df4eb7
      Mahatthana (Kelvin) Nomsawadi authored
      Follow-up milestone 2 for [Epic] Re-design transparent theme for static embedding and public links (#46491)
      
      * Update new Metabase base colors
      
      * Make hover and selected state consistency
      
      * Fix wrong embed frame border color
      
      * Refactor: extract props
      
      * Fix border color
      
      * Fix pivot table cell hover color
      
      * Fix smart scalar text contrast
      
      * Add smart scalar tests + update text color design
      
      * Theme dashboard card menu
      
      * Remove dashcard action menu tests since they don't work in loki
      
      * Fix storybook background color for dark theme
      
      * Update loki snapshots
      
      * Disable untestable tests
      
      * Make sure all storybook inherit all necessary base styles
      
      * Skip untestable tests
      
      * Update loki snapshots
      Unverified
      a4df4eb7
    • Mahatthana (Kelvin) Nomsawadi's avatar
      Fix unintentional color changes (#46516) · 6fb1e51d
      Mahatthana (Kelvin) Nomsawadi authored
      * Sync storybook colors with the main app
      
      * Fix Metabase badge color
      
      * Fix wrong mixed brand colors
      
      * Add mistakenly removed colors
      
      * Update comments
      
      * Correct `brand-light` color
      
      * Fix wrong Mantine button color when passing `color` prop
      
      * Update loki snapshots
      Unverified
      6fb1e51d
  19. Aug 07, 2024
  20. Aug 05, 2024
    • Mahatthana (Kelvin) Nomsawadi's avatar
      [Milestone 2] Re-design transparent theme for static embedding and public links (#46428) · 3d5233bc
      Mahatthana (Kelvin) Nomsawadi authored
      
      * Update popover dark theme on part of ParameterFieldWidget (#45813)
      
      * Add filter visual tests
      
      * Simplify embed CSS colors
      
      * [WIP] theme filter popover
      
      * Move theme to global node, so it affects portal too
      
      * Fix embed colors
      
      * Theme part of ParameterFieldWidget
      
      * Add story for parameter with dropdown
      
      * Update loki snapshots
      
      * Update more popover style
      
      * Update loki snapshots
      
      * Remove outdated snapshots
      
      * Update loki snapshots with production build
      
      * Fix loki not working with `userEvent`
      
      * Theme mantine popover
      
      * Improve dark theme text-brand visibility
      
      * Update loki snapshots from master change
      
      * Fix new colors not working in SDK
      
      * Fix flaky Loki tests
      
      * Filter popover dark theme part 2 (#46284)
      
      * Theme Date all options
      
      * wip add month and year filter
      
      * Fix wrong CSS custom property declaration
      
      * Test hover state
      
      * Update Loki snapshots
      
      * Theme date month and year
      
      * Add quarter and year tests
      
      * Theme date single and date range
      
      * Style relative date
      
      * Update frontend/src/metabase/admin/datamodel/components/filters/pickers/DatePicker/DatePickerHeader.styled.tsx
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      ---------
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      * Complete theming filter popover night mode (#46365)
      
      * Style unit of time and number widget
      
      * Attempt to fix flaky Loki snapshots
      
      * Address white background checkbox on dark theme
      
      * Style search filter single value
      
      * Fix mis color
      
      * Update snapshots
      
      * Remove duplication on play functions since they're long
      
      * Milestone 2 polishing (#46382)
      
      * Matches the footer padding with header padding
      
      * Update snapshots
      
      * Add E2E to prevent further problem
      
      * Try to reduce flaky Loki tests
      
      * Remove unused color shades (#46421)
      
      * Update no results illustration to work better on dark theme
      
      * Update loki snapshots
      
      ---------
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      Unverified
      3d5233bc
    • Kamil Mielnik's avatar
      Update "Clear" and "Reset" buttons (#46050) · 35d9a817
      Kamil Mielnik authored
      * Refactor dashboard helpers to TypeScript
      
      * Reuse existing helper
      
      * Refactor cy.createQuestionAndDashboard to a function helper
      - there was no good place to put it, so I also created new helpers/api directory
      - and I moved all helpers using cy.request (with 1 exception, where a CSV download helper also does a bunch of assertions) in there
      
      * Merge TS command definitions into a single file
      
      * Export types
      
      * Make dashboardDetails optional
      
      * Export StructuredQuestionDetails
      
      * Remove duplicated function
      
      * Extract createNativeQuestion to separate file
      - 1 exported helper per file, no exceptions!
      
      * Reuse const
      
      * Add test for unit of time parameters
      
      * Improve assertions
      
      * Extract createDashboardWithParameters
      
      * Add tests for time parameters
      
      * Fix hardcoded parameter_mappings ids
      
      * Format code
      
      * Extract checkButtonVisible
      
      * Improve test names
      
      * Add tests for location parameters
      
      * Reuse const
      
      * Update test name
      
      * Add WidgetStatusButton
      
      * Add unit tests for WidgetStatusButton
      
      * Rename handler
      
      * Remove redundant assertions
      
      * Add iconSize prop
      
      * Fix styling
      
      * Replace WidgetStatusIcon with WidgetStatusButton
      - Remove WidgetStatusIcon
      
      * Fix tooltip issue
      
      * Fix tooltip for good
      
      * Use default compact button size
      
      * Prevent overflow
      
      * Fix alignment issues in parameter sidebar
      - Remove iconSize prop
      - Simplify StatusConfig
      
      * Update padding in Unit of Time input to make it consistent with other inputs in dashboard parameter sidebar
      
      * Update unit tests
      
      * Rename WidgetStatusButton to WidgetStatus
      
      * Explicitly specify what is a button
      
      * Update helpers in tests
      
      * Update when buttons are displayed according to the new spec, update tests
      
      * Simplify label
      
      * Remove utils
      
      * Adjust padding
      
      * Update reset icon
      
      * Make the button gray when color is undesired
      
      * Remove old definitions
      
      * Add missing export
      
      * Remove unused type
      
      * Add test for location (single value)
      
      * Add test for id (single value)
      
      * Add more assertions
      
      * Add test for id (multiple values)
      
      * Improve name
      
      * Add tests for number (single and multi value)
      
      * Add tests for text (single and multi value)
      
      * Rename file
      
      * Update comment
      
      * Update test name
      
      * Use unique ids
      
      * Use new color name
      
      * Add a test for chevron icons alignment
      
      * Lift className up
      
      * Make types more specific
      
      * Sort attributes
      
      * Revert "Make types more specific"
      
      This reverts commit 2e6994b5.
      
      * Revert "Use new color name"
      
      This reverts commit 2cc68df6.
      
      * Remove TODO
      
      * Introduce editFilter util, refactor chevron assertions to work with any number of chevrons
      
      * Connect label to the default value input in ParameterSettings
      
      * Add a whitespace in the default value label when required "(required)" is shown
      
      * Add parameter sidebar tests for unit of time
      
      * Add parameter sidebar tests for date filters
      
      * Extract checkParameterSidebarDefaultValue
      
      * Add parameter sidebar tests for location - single values
      
      * Add parameter sidebar tests for location - multiple values
      
      * Use constants for test case names
      
      * Add parameter sidebar tests for id parameters - single values
      
      * Add parameter sidebar tests for id parameters - multi values
      
      * Add parameter sidebar tests for number parameters - single values
      
      * Add parameter sidebar tests for number parameters - multi values
      
      * Add parameter sidebar tests for text parameters - single values
      
      * Add parameter sidebar tests for text parameters - multi values
      
      * Extract checkDashboardFilters
      
      * Use checkDashboardFilters in time parameters
      
      * Use checkParameterSidebarDefaultValue in checkDashboardFilters
      
      * Use checkDashboardFilters everywhere
      
      * Sort helpers
      
      * Rename checkDashboardFilters to checkDashboardParameters
      
      * Rename checkOnlyOneButtonVisible to checkStatusIcon
      
      * Add test suite for sql filters, coverage for text filter - single value
      
      * Add a whitespace in the default value label when required "(required)" is shown - SQL filters
      - Similar to 930dd3a5
      
      * Add test suite for sql filters, coverage for number filters
      
      * Add test for sql filters - date
      
      * Remove unused things, rename placeholder to labelOrPlaceholder
      
      * Apply label only to 1 input
      
      * Add test for sql filters - field parameters
      
      * Fix date tests
      
      * Make all sql filters tests work
      
      * Add assertions for required without default in text filters
      
      * Add assertions for required without default in date filters
      
      * Add assertions for required without default in field filters
      
      * Allow clearing required SQL filters without default value
      
      * Add a test to ensure comma-separated numeric values work
      
      * Update unit test
      
      * Update helper
      
      * Update e2e tests
      
      * Use != null and == null checks with defaultValue instead of just checking if its truthy/falsy
      
      * Run `yarn test-visual:loki` and then `yarn loki approve`
      - see https://metaboat.slack.com/archives/C505ZNNH4/p1721289287037069
      
      * Use parameterHasNoDisplayValue to account for empty strings
      
      * Run `yarn test-visual:loki` and then `yarn loki approve` after merging master
      - see https://metaboat.slack.com/archives/C505ZNNH4/p1721289287037069
      
      * Run `yarn test-visual:loki` and then `yarn loki approve`
      - see https://metaboat.slack.com/archives/C505ZNNH4/p1721289287037069
      
      * Run `yarn test-visual:loki` and then `yarn loki approve`
      - see https://metaboat.slack.com/archives/C505ZNNH4/p1721289287037069
      Unverified
      35d9a817
    • Emmad Usmani's avatar
      add both option for pie chart percentages (#46399) · 6eaff50e
      Emmad Usmani authored
      * add both option for pie chart percentages
      
      * create story snapshot
      Unverified
      6eaff50e
  21. Aug 02, 2024
  22. Jul 31, 2024
    • Emmad Usmani's avatar
      migrate pie chart to echarts (#43555) · 4ec1fd88
      Emmad Usmani authored
      
      * echarts pie
      
      * setup rendering pipeline for static pie chart (#43547)
      
      * setup rendering pipeline for static pie chart
      
      * remove more donut code from BE
      
      * compute viz settings for static pie chart (#43548)
      
      * compute viz settings for static pie chart
      
      * create chartModel for pie chart (#43549)
      
      * create chartModel for pie chart
      
      * render static pie chart (#43550)
      
      * render static pie chart
      
      * render static pie chart legend (#43551)
      
      * disable animation
      
      * alter static-viz pie chart be tests to account for echarts-impl
      
      The old implementation had a different implementation for legends, separate from the 'categorical/donut' js
      render. Echarts nicely combines the legend into the pie render, so we don't need to worry about that stuff anymore.
      
      Instead, I've written a simple set of assertions on the echarts pie implementation now.
      
      * Get rid of this test that was attempting to use old code path
      
      * echarts pie
      
      * One last spot where we don't need old pie stuff anymore.
      
      * add loki specs
      
      * save snapshots
      
      * fix small slices disappearing
      
      * update snapshots
      
      * add repro story for exteremely small slice
      
      * add snapshot
      
      * fix rounded total
      
      * fix percent formatting in legend
      
      * fix date dimension formatting
      
      * fix binned and relative date dimension labels on legend
      
      * add specs and snapshots for null and unaggregated dimensions
      
      * hide overlaping labels on chart
      
      * fix null dimension color
      
      * show error when metric column has a negative value
      
      * move legend to botom
      
      * make dot margin smaller
      
      * add columns to legend
      
      * sort legend vertically
      
      * dont use grid if only one row for legend
      
      * never hide legend in static viz
      
      * fix empty column culling
      
      * show single legend item
      
      * add long dimension name story
      
      * fix long dimension name in legend
      
      * right justify percentages in legend
      
      * remove negative error and re-add stories
      
      * handle negative values
      
      * create chart definition for dynamic pie chart
      
      * render chart with responsiveness
      
      * add comment for borderWidth calculation
      
      * center legend
      
      * fix legend centering
      
      * add story for missing currency formatting
      
      * fix missing currency formatting
      
      * fix graalvm crashing
      
      * fix column settings computation
      
      * add another currency formatting repro
      
      * fix wrong order of col settings computations
      
      * convert colors to hex
      
      * fix colors missing from viz settings
      
      * temp todo comment
      
      * fix colors in dynamic viz
      
      * update snapshots
      
      * fix crashing due to invalid dimension/metric setting
      
      * fix chart crashing for boolean dimension with labels on chart
      
      * add repro
      
      * fix regression in label formatting
      
      * fix labels on small slices
      
      * update specs
      
      * emphasis state for slice hover
      
      * recreate old label hiding logic
      
      * include height measurement in getIsLabelVisible
      
      * move d3 slices to chart model
      
      * fix labels on static viz
      
      * wip basic hover state and tooltip
      
      * fix emphasis style getting stuck by memoizing everything
      
      * update total display value
      
      * fix stuck emphasis by setting notMerge to false for pie only
      
      * handle other slice in tooltip
      
      * fix bugs in total graphic
      
      * use OTHER_SLICE_KEY const in static viz legend
      
      * implement legend and connect to hover state
      
      * implement click actions
      
      * fix slice not clickable
      
      * handle all zero slices
      
      * add repro
      
      * update specs
      
      * update pie_chart.cy.spec.js
      
      * update chart_drill.cy.spec.js
      
      * add repro for single column legend
      
      * use minimum 2 legend columns
      
      * fix legend text overlap
      
      * use more margin for single row legend
      
      * update specs
      
      * add repro for 28568
      
      * add repro for 38424
      
      * fix percent label not showing on large slices
      
      * hide total text when chart is small
      
      * replace comment about animations
      
      * show warning for mixed positive and negative values
      
      * delete old chart files
      
      * fix type errors
      
      * update d3 import
      
      * update specs
      
      * remove branch from uberjar.yml
      
      * dont put single slice into other
      
      * fix lint errors and pie render test
      
      * update more d3 imports
      
      * remove unused settings
      
      * revert preview-card-width change
      
      * remove unused import
      
      * remove majorWidth option from formatting
      
      * use debounceLeading refresh mode in ChartWithLegend
      
      * remove TOTAL_GRAPHIC_OPTION constant
      
      * delete unit test for pie chart
      
      * fix failing test by doing set comparison instead
      
      * fix typo in comment
      
      * remove SUNBURST_SERIES_OPTION constant
      
      * fix lint by removing unused eslint-disable
      
      * move default color computation to getValue
      
      * fix static pie test to show that we will always render the legend
      
      * update pie border color for night mode and embedding
      
      * fix styles for embedding case and add stories
      
      * skip embedding stories
      
      ---------
      
      Co-authored-by: default avatarAdam James <adam.vermeer2@gmail.com>
      Unverified
      4ec1fd88
  23. Jul 17, 2024
    • Mahatthana (Kelvin) Nomsawadi's avatar
      [Milestone 1] Re-design transparent theme for static embedding and public links (#45633) · ffa54f0f
      Mahatthana (Kelvin) Nomsawadi authored
      
      * refactor: Simplify code (#44275)
      
      * Remove dead code
      
      * Remove values same as default options
      
      * Simplify hook API: Removing unused argument
      
      * review: Remove redundant code
      
      * Update look and feel copy (#44277)
      
      * Update static embed tab name
      
      * Update copies + rearrange settings
      
      * Use Mantine Select
      
      * Fix E2E tests by applying new copies
      
      * Fix unit tests
      
      * Fix more E2E tests
      
      * review: Add a comment on Mantine `value` quirk
      
      * Adjust spacing and relevant tests
      
      * Move `transparent` theme to its own toggle option (#44362)
      
      * Reorder the function to match the visual order
      
      * Make function name consistent
      
      * Introduce the new background option (do nothing yet)
      
      * Remove redundant exports
      
      * Tighten up values to reduce confusion
      
      * Make the parameters order consistent
      
      * Only show background option on dashboads (spec)
      
      * Allow dashboard embed background to be transparent
      
      * Make filter background and header/footer transparent
      
      * Fix type error
      
      * Fix unit tests
      
      * Fix E2E tests
      
      * Fix checkerboard pattern not showing when no background
      
      * Add unit tests
      
      * Add missing copy update
      
      * Remove remnants from outdated solutions within the PR
      
      * Improve code comment references
      
      * Fix E2E tests since the last copy change
      
      * Fix type errors from merging master
      
      * Update light theme style (#44488)
      
      * Add background to dashboard filter on light theme
      
      * Make position sticky works with transparent background
      
      * Fix sticky filters not working on theme=transparent
      
      * Fix unit tests
      
      * Simplify IntersectionObserver components + fix E2E tests
      
      * Add storybook for embedded dashboard
      
      * Clarify the use of ExplicitSize context
      
      * Remove unused code
      
      * Formatting
      
      * Review: Better mock `isWithinIframe`
      
      * Update dark theme to match the new design (#45134)
      
      * Update light theme checkerboard colors
      
      * Update checkerboard colors
      
      * Consolidate dashboard light/dark border style
      
      * Use `--mb-color-text-primary` for light theme dashboard
      
      * Update embed dashboard dark theme primary color
      
      * Migrate dashboard text-secondary color
      
      * Migrate text-tertiary color
      
      * Migrate gold, axis, and split line to CSS custom properties
      
      * Fix failed e2e tests
      
      * Fix static viz not working
      
      * Update loki snapshots
      
      * Remove unnecessary CSS custom property
      
      * Make diff more consistent
      
      * Address review: reusing type
      
      * Support backward compatibility when passing `theme=transparent` (#45214)
      
      * Match preview query with the actual code query
      
      * Handle backward compatibility when `theme=transparent` on dashboard
      
      * Add unit tests
      
      * Address review: Make tests less repetitive
      
      * Update Metabase banner upsell UI (#45424)
      
      * Prepare folder for nested components
      
      * Update upsell UI
      
      - add `fullWidth` variant
      - align title to the left
      
      * Fix Upsell banner having horizontal scroll on full width
      
      * Add new Metabase banner upsell component
      
      * Fix unit tests
      
      * Address review: Fix typo
      
      Co-authored-by: default avatarRyan Laurie <30528226+iethree@users.noreply.github.com>
      
      * Address review: Move upsel UI to `admin/upsells`
      
      Co-authored-by: default avatarRyan Laurie <30528226+iethree@users.noreply.github.com>
      
      * Remove unnecessary ESLint disable comment
      
      We can remove it since the file is in admin folder, and is already excluded from the rule.
      
      * Address review: remove default value property
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      * Address review: Extract magic number
      
      ---------
      
      Co-authored-by: default avatarRyan Laurie <30528226+iethree@users.noreply.github.com>
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      * More UI updates (#45450)
      
      * Update dark theme text colors to match the new design
      
      * Update powered by metabase logo
      
      * Update loki snapshots
      
      * Fix unit tests
      
      * Fix formatting
      
      * Fix wrong dark theme metabase logo badge color
      
      * Address review: Compress SVG with svgo
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      * Fix E2E tests
      
      * Update loki snapshots
      
      * Clarify Metabase base colors with comments
      
      ---------
      
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      
      * Add more tests (#45628)
      
      * Add static embed question stories
      
      * Add loki snapshots
      
      * Add transparent theme for static dashboard tests
      
      * Add transparent theme tests for static questions
      
      * Assert `background` option in analytics tests
      
      * Remove tests for logic that has been moved
      
      * Update loki snapshots
      
      * Fix failed E2E tests from merging master
      
      ---------
      
      Co-authored-by: default avatarRyan Laurie <30528226+iethree@users.noreply.github.com>
      Co-authored-by: default avatarDenis Berezin <denis.berezin@metabase.com>
      Unverified
      ffa54f0f
  24. Jul 12, 2024
  25. Jul 11, 2024
  26. Jul 09, 2024
  27. Jul 08, 2024
  28. Jul 01, 2024
Loading