[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-02-14 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-365841957
 
 
   @shyam2794 Hi, this PR is not merged yet, and if you would like to have a 
try, you have to cherry pick this PR manually by yourself, then you have to 
recompile the frontend assets.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-02-12 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-365168344
 
 
   HI, all, we have updated lots of screenshots in 
   https://github.com/apache/incubator-superset/pull/4394, please vote for us, 
thanks.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-02-07 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-363745069
 
 
   We're almost done.
   
   @WangXueminGit is really working quite hard on this feature and after 
internal code review, we'll push this to upstream soon.
   
   Here's a small demo of our work:
   
   ![ezgif 
com-resize](https://user-images.githubusercontent.com/1164623/35914802-9a8c8442-0c3f-11e8-8806-6639c5a1b77d.gif)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-02-07 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-363745069
 
 
   We're almost done.
   
   @WangXueminGit is really working quite hard on this feature and after 
internal code review, we'll push this to upstream.
   
   Here's a small demo of our work:
   
   ![ezgif 
com-resize](https://user-images.githubusercontent.com/1164623/35914802-9a8c8442-0c3f-11e8-8806-6639c5a1b77d.gif)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-01-27 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-360985412
 
 
   @kartikshanbhag oh really? we think the second one looks more aligned 
considering layout, while the first one looks a little messy about layout.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2018-01-15 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-357839164
 
 
   @kartikshanbhag we're working on it from this week. It should be PRed by 
the end of this month.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2017-12-26 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-354039432
 
 
   @shyam2794 we will start to do this from Jan, so be patient!


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2017-12-22 Thread GitBox
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-353571580
 
 
   @shyam2794 Right now, no. We're considering contribute our patch to 
upstream, maybe start from Next January.
   
   And to give you a sneak:
   
   
![image](https://user-images.githubusercontent.com/1164623/34295231-34f69fae-e747-11e7-866e-f9af4aba0d1b.png)
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2017-09-11 Thread git
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-328480116
 
 
   Hi, @gangh 
   
   Right now these feature is hosted on our own internal private codebase, and 
we're considering to contribute back this feature to upstream in one or two 
months.
   
   I wrote our work here because I was wondering how many people are interested 
in this kind of feature.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables

2017-09-07 Thread git
xiaohanyu commented on issue #3416: Add ability to color cells on Pivot Tables
URL: 
https://github.com/apache/incubator-superset/issues/3416#issuecomment-327708132
 
 
   Hi, all,
   
   Actually, we have implement similar features for pivot table and plain 
table, in our own branch.
   
   Some screenshot here:
   
   The final table style:
   
   
![image](https://user-images.githubusercontent.com/1164623/30150439-fba16796-93dd-11e7-8059-6f58a0642f11.png)
   
   Configure table column style based on some rules (like if the value is 
greater than 100%, then set the cell color to red)
   
   
![image](https://user-images.githubusercontent.com/1164623/30150466-1755f952-93de-11e7-90c8-bee537fcc5f0.png)
   
   Configure table row style based on some rules (like if the row contains some 
keyword, then highlight the whole row):
   
   
![image](https://user-images.githubusercontent.com/1164623/30150496-3bc4aaae-93de-11e7-9344-d5191d432511.png)
   
   I was wondering if anybody have interest in this feature. We can consider to 
contribute back to upstream.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services