Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor dashboard chart html, make several css improvements. #175

Merged
merged 2 commits into from
Mar 14, 2016

Conversation

williaster
Copy link
Contributor

Fixes:

  • Refactor dashboard chart html (some invalid elements used previously, e.g., <nobr>
  • Improve dashboard chart header (emphasize with color, wrap long titles, move controls to new line)
  • Add css rule for nvd3 tooltips to emphasize (nick h. request)
  • Ensure nvd3 tooltip is visible when chart overflows (lindsay request)

slice.render();
}
});
if (immune) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was throwing errors

@@ -95,7 +98,9 @@ var Dashboard = function (dashboardData) {
enabled: true,
stop: function (e, ui, element) {
var slice_data = $(element).data('slice');
dashboard.getSlice(slice_data.slice_id).resize();
if (slice_data) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was throwing errors also

@mistercrunch
Copy link
Member

👍 !!!

williaster added a commit that referenced this pull request Mar 14, 2016
refactor dashboard chart html, make several css improvements.
@williaster williaster merged commit e1eb236 into master Mar 14, 2016
@williaster williaster deleted the chris/css-fixes branch March 14, 2016 06:03
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this pull request Oct 4, 2021
* feat: add fallback component

* feat: add superchart shell

* feat: add vx/responsive type declaration

* fix: path and dependencies

* test: add unit tests

* test: add more tests

* docs: add storybook

* test: fix FallBackComponent test

* feat: make fallback accepts width and height

* test: reach 100%

* fix: test

* fix: add more checks

* refactor: rename SuperChartKernel to SuperChartCore

* refactor: separate backward-compatibility code into another wrapper
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 17, 2021
* feat: add fallback component

* feat: add superchart shell

* feat: add vx/responsive type declaration

* fix: path and dependencies

* test: add unit tests

* test: add more tests

* docs: add storybook

* test: fix FallBackComponent test

* feat: make fallback accepts width and height

* test: reach 100%

* fix: test

* fix: add more checks

* refactor: rename SuperChartKernel to SuperChartCore

* refactor: separate backward-compatibility code into another wrapper
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 17, 2021
* feat: update type for line chart series

* fix: another place
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 24, 2021
* feat: add fallback component

* feat: add superchart shell

* feat: add vx/responsive type declaration

* fix: path and dependencies

* test: add unit tests

* test: add more tests

* docs: add storybook

* test: fix FallBackComponent test

* feat: make fallback accepts width and height

* test: reach 100%

* fix: test

* fix: add more checks

* refactor: rename SuperChartKernel to SuperChartCore

* refactor: separate backward-compatibility code into another wrapper
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 24, 2021
* feat: update type for line chart series

* fix: another place
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 25, 2021
* feat: add fallback component

* feat: add superchart shell

* feat: add vx/responsive type declaration

* fix: path and dependencies

* test: add unit tests

* test: add more tests

* docs: add storybook

* test: fix FallBackComponent test

* feat: make fallback accepts width and height

* test: reach 100%

* fix: test

* fix: add more checks

* refactor: rename SuperChartKernel to SuperChartCore

* refactor: separate backward-compatibility code into another wrapper
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 25, 2021
* feat: update type for line chart series

* fix: another place
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 26, 2021
* feat: add fallback component

* feat: add superchart shell

* feat: add vx/responsive type declaration

* fix: path and dependencies

* test: add unit tests

* test: add more tests

* docs: add storybook

* test: fix FallBackComponent test

* feat: make fallback accepts width and height

* test: reach 100%

* fix: test

* fix: add more checks

* refactor: rename SuperChartKernel to SuperChartCore

* refactor: separate backward-compatibility code into another wrapper
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 26, 2021
* feat: update type for line chart series

* fix: another place
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.8.1 labels Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.8.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants