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.

Sign up for GitHub

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

Jump to bottom

Fix y axis on graphs with %s #114

Closed
nhuber opened this issue Jan 14, 2016 · 3 comments
Closed

Fix y axis on graphs with %s #114

nhuber opened this issue Jan 14, 2016 · 3 comments

Comments

@nhuber
Copy link

nhuber commented Jan 14, 2016

When you have a graph with y axis between 0 and 1 it appears to show them in "m"

screen shot 2016-01-14 at 11 22 28 am

All of these should really be their "m" value / 10 and in percent. So the top line is in reality values of around 6%
@mistercrunch
Copy link
Member

Yeah, the m for "micro" is confusing. The solution to that is using the d3 Y axis formatting in the form. It's not super intuitive but it's very flexible. Maybe I should have a free form dropdown with common suggestions instead of the current free form field:
screen shot 2016-01-14 at 11 44 56 am

@nhuber
Copy link
Author

nhuber commented Jan 16, 2016

Perfect. Maybe just make % the default for <1 values or something

@mistercrunch
Copy link
Member

Fixed.

kristw added a commit to kristw/incubator-superset that referenced this issue Sep 19, 2018
@kristw
Merge pull request apache#114 from kristw/kristw-cherry-sep18
901ea8a 
Cherry-picks: Several bug fixes and react HMR
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this issue Oct 4, 2021
@kristw
feat: add mergeMargin() function (apache#114)
c558938 
* feat: add mergeMargin()

* fix: typings

* refactor: address chris' comments
zhaoyongjie referenced this issue in zhaoyongjie/incubator-superset Nov 17, 2021
@kristw @zhaoyongjie
feat: add mergeMargin() function (#114)
4852719 
* feat: add mergeMargin()

* fix: typings

* refactor: address chris' comments
zhaoyongjie referenced this issue in zhaoyongjie/incubator-superset Nov 24, 2021
@kristw @zhaoyongjie
feat: add mergeMargin() function (#114)
feb14be 
* feat: add mergeMargin()

* fix: typings

* refactor: address chris' comments
zhaoyongjie referenced this issue in zhaoyongjie/incubator-superset Nov 25, 2021
@kristw @zhaoyongjie
feat: add mergeMargin() function (#114)
747368c 
* feat: add mergeMargin()

* fix: typings

* refactor: address chris' comments
zhaoyongjie referenced this issue in zhaoyongjie/incubator-superset Nov 26, 2021
@kristw @zhaoyongjie
feat: add mergeMargin() function (#114)
e5d7e50 
* feat: add mergeMargin()

* fix: typings

* refactor: address chris' comments
@snyk-bot snyk-bot mentioned this issue May 20, 2022
Open
@snyk-bot snyk-bot mentioned this issue Nov 4, 2022
Open
@snyk-bot snyk-bot mentioned this issue Jun 12, 2023
Open
This was referenced Jul 27, 2023
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mistercrunch @nhuber