Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

[chart plugin]TableVis Plugin #66

Closed
wants to merge 1 commit into from
Closed

[chart plugin]TableVis Plugin #66

wants to merge 1 commit into from

Conversation

conglei
Copy link
Contributor

@conglei conglei commented Dec 14, 2018

馃挃 Breaking Changes

馃弳 Enhancements
This is to bring tablevis as a plugin.

馃摐 Documentation

馃悰 Bug Fix

馃彔 Internal

@conglei conglei added #enhancement New feature or request WIP Work in progress labels Dec 14, 2018
@conglei conglei requested a review from a team as a code owner December 14, 2018 01:03
import $ from 'jquery';
import PropTypes from 'prop-types';
import dt from 'datatables.net-bs';
import 'datatables.net-bs/css/dataTables.bootstrap.css';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we may not be able to import this type of file in babel.

if we refactor datatables and still need to import css, we could instead copy it in the lib output, and have incubator superset import (since it has proper css loader support)

@conglei conglei closed this Mar 25, 2019
@conglei conglei deleted the conglei_table branch March 25, 2019 03:12
kristw added a commit that referenced this pull request Apr 17, 2020
* fix: fallback to default margin when margin is partially set

* feat: can disable axis title

* feat: adjust margin according to axis title visibility

* feat: include margin in formData

* feat: add buildQuery

* fix: address kyle comments

* fix: remove string false
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
#enhancement New feature or request WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants