Open MuhammedAlkhudiry opened 4 years ago
Thanks for your awesome work~
I think moving the table
is useful for most users. I am glad to add this feature in quill-better-table
, can you provide me a sandbox or a demo in codepen? Or PR welcome.
About styling the table
, I can not imagine the final effect. A demo will help me have a clearer understanding.
About popper.js
, I think keep less dependences is also important.
Keep in touch~ PR is a better place for these.
Thanks for your awesome work~ I think
moving the table
is useful for most users. I am glad to add this feature inquill-better-table
, can you provide me a sandbox or a demo in codepen? Or PR welcome. Aboutstyling the table
, I can not imagine the final effect. A demo will help me have a clearer understanding. Aboutpopper.js
, I think keep less dependences is also important. Keep in touch~ PR is a better place for these.
Thanks for your awesome module!
I'm sorry, but I was working in production file which is this file https://github.com/soccerloway/quill-better-table/blob/master/dist/quill-better-table.js, since I did not expect to change alot, so showing it in CodePen is a pain. However, I'm planning to use the source file for the future (I will work for row-height changing tool, and an option to equalize the size of each cell, and others, don't wait for me if you want to implement those :) )
About styling the table
, these images my help:
For background color
For border style
No image needed for the rest
Since you are here, please try to solve these two problems in column-width changing tool:
Pretty UI~ I can not wait to receive your contribution now. You can send me PRs in your free time.
I will try to add the features you mentioned for quill-better-table
gradually in my free time.
work for row-height changing tool
I have implemented row-height changing tool before, row-height was affected by contents, it will call blots' optimize
function many times, have a low performance. So I canceled this feature. Maybe automatic row-height is enough to use. It's my personal opinion~:)
Changing a column width in should not change the width of the table (try changing a column width in quill then in Google Docs to understand)
In quill-better-table
, the width of table will change by column width. Since the table may be displayed in a small area, I decide to wrap table in a scrollable div
. It is good for displaying table in anywhere. So I think the width of table is necessary.
@MuhammedAlkhudiry I am waiting for your contributions~~~Thanks a lot:)
@KMLX28 could you please give further info on how to integrate your above changes with current quill-better-table master branch? Am interested in trying, but not sure where to include your changes
These feature changes for moving table are not yet merged with master right?
I've added in the basic plumbing for this in this commit in my fork, if anyone wants to play with it or use it as the basis for a pull request.
https://github.com/arimus/quill-better-table/commit/48bdead3dc675e0ffb4876fd6cd3334366774e17
It mostly works with some quirks here and there. Thanks @KMLX28 for the basics. I had to adjust a decent amount, but for the most part it's working pretty well.
I added the ability for:
Note: I tried to follow your way of making a tool as much as possible.
First: moving the table tool class:
Second: stying the table tool class:
Also, I added new function the module which is getSelectedCells();
I also, added style for these tool, and modify the style for existed tools:
Note: I use popper.js for positioning the tools, you can remove it or consider using it.
feel free to take and modfiy my code as needed.
I'm sorry if issues page is not the place for these kind of posts.