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

Table cell/row/column selection – stage I #3202

Closed
20 of 21 tasks
Reinmar opened this issue Jun 18, 2018 · 7 comments
Closed
20 of 21 tasks

Table cell/row/column selection – stage I #3202

Reinmar opened this issue Jun 18, 2018 · 7 comments
Labels
Epic package:table type:feature This issue reports a feature request (an idea for a new functionality or a missing option).

Comments

@Reinmar
Copy link
Member

Reinmar commented Jun 18, 2018

We started to work on the table selection feature. Since it is a bigger task that will be delivered over multiple releases we split the task to the following sub-tickets:

MVP (Stage I)

Stage II

Tracked here: #6285.

Stage III (optional features)

Tracked here: #6297.


Right now, as defined in https://github.com/ckeditor/ckeditor5-table/issues/3 we support only "a custom table selection". This means that it's possible to place the selection inside table cells, but you cannot select an entire table cell and, most importantly, more than one cell at a time. Which means that you cannot select rows and columns.

CKEditor 4 implements a custom cell/row/col selection (you check it live e.g. on https://ckeditor.com/ckeditor-4/#document). It works incomparably better than the native selection which in all browsers (without maybe Firefox) is just useless.

What we need is something very similar to what CKEditor 4 offers.

@jodator jodator self-assigned this Aug 2, 2018
@jodator
Copy link
Contributor

jodator commented Aug 2, 2018

Some POC of fake selection in tables with current post fixers, table model (with block content), etc:

peek 2018-08-02 16-05

The above POC is implemented as custom class wchich tracks selection on table in three different modes:

  1. block
  2. column
  3. row

I have some UI/UX questions (cc @oleq, @dkonopka) on how we're going to implement this.
In https://github.com/ckeditor/ckeditor5-table/issues/1#issuecomment-392034926 @oleq defined how merge cell option should behave. But do we fancy another modes besides "block" mode? I've implemented "row"/"column" modes as it was simple addition to the "block" mode. Also I've seen this on other text-edtiors (like LibreOffice):

peek 2018-08-02 16-24

ps.: Side note - right now I don't have strong opinion/idea how to translate this fake selection to commands as such selection is not possible in the model due to selection post-fixer. Right now I'm leaning towards TableSelection plugin which might be used by the TableToolbar to determine which buttons/options render since with multple cell selection some options may be changed, like:

  • Remove row -> Remove rows
  • Insert row below -> Insert 3 rows below (ie when 3 rows are already selected)

@oleq
Copy link
Member

oleq commented Aug 2, 2018

The merge cells button should be a simple button that merges selected cells, nothing more.

As for the column/block selection feature, I think we can live without it (ATM). Let's talk about it and find out if it makes sense for us in the follow-up issue, OK?

@deepak-khopade-isdr
Copy link

When would this feature be available for tables?

@harshinim
Copy link

Do we know when this feature would be available? I have a requirement where users want to paste to part of the table from excel.

@jodator jodator removed their assignment May 22, 2019
@mlewand mlewand transferred this issue from ckeditor/ckeditor5-table Oct 9, 2019
@mlewand mlewand added this to the nice-to-have milestone Oct 9, 2019
@mlewand mlewand added status:confirmed type:feature This issue reports a feature request (an idea for a new functionality or a missing option). package:table labels Oct 9, 2019
@jodator
Copy link
Contributor

jodator commented Oct 14, 2019

The branches with a preview were updated to i/3202 in ckeditor5-table and ckeditor5-engine repos. Old branches (t/63) were removed.

@jodator
Copy link
Contributor

jodator commented Nov 5, 2019

Would be nice to have this handled by the inspector: ckeditor/ckeditor5-inspector#68.

@jodator jodator added the Epic label Jan 22, 2020
@Reinmar Reinmar changed the title Implement table cell/row/column selection Table cell/row/column selection – stage I Feb 19, 2020
@jodator jodator modified the milestones: nice-to-have, iteration 30 Feb 21, 2020
@Reinmar
Copy link
Member Author

Reinmar commented Mar 12, 2020

Done :) It can be tested in the nightly docs: https://ckeditor5.github.io/docs/nightly/ckeditor5/latest/features/table.html

@Reinmar Reinmar closed this as completed Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic package:table type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Projects
None yet
Development

No branches or pull requests

6 participants