Conversation
prabhjyotsingh
commented
Dec 4, 2015

b48f582 to
f86db0a
Compare
|
We already have one to resize the height of the paragraph result. |
|
Yes, I've noticed it, but it only shows up when there is a table/graph in view. Should I try merge it here, or this there? |
|
It's not too shocking to have both resize handles, however the resize graph height doesn't work anymore. |
f86db0a to
1358535
Compare
|
I have tested and it works really well. Thanks @prabhjyotsingh for really improvement. One thing is, resize drag handle is shown in report mode. But i think it more sense to hide there. What do you think? |
|
Could you also un-commit the note.json? |
1358535 to
b15e4cf
Compare
|
Thanks for the review and finding bugs;
|
|
Does this looks ok ? |
|
@prabhjyotsingh I will test it again tomorrow, sorry for the delay |
b15e4cf to
39184f6
Compare
|
Thanks @corneadoug for the feedback and a providing me with a lot of test cases. |
b403ead to
8ce2abb
Compare
8ce2abb to
bdc3f30
Compare
|
Tested and LGTM. |
|
Merge if there're no more discussions |
|
Re-tested, +1 |






