[Okular-devel] [Bug 279859] table selection tool - new feature (patch enclosed)

[Jiri Baum]

Hello,

Albert Astals Cid:
> Aboutdata is probably the easiest place, you can also add
> your name to the headers if you feel like it.

Thanks.

> Ignoring the drm is a no go, please do what the rest of the code does.

OK, I'll do that. I guess that always was going to go in pretty soon, so 
better do it now.

> If you prefer we have git.reviewboard.kde.org that sometimes helps a bit
> with patch reviewing.

Thanks, I may post it there.

> Also i'd prefer if you tried to fix the problem you have with Esc.

Do you have any suggestions for that, please? As you can see, I've got 
Qt::Key_Escape and  Qt::Key_C doing the same functionality, but for some 
reason the Esc doesn't seem to get there.

> Finally i'm not sure this is not a extra specialized feature and that it
> really makes sense to add to okular itself. Can you please describe the
> use case?

I imagine it's a fairly common requirement...? Many documents contain tables, 
and being able to copy'n'paste them while preserving the row/column structure 
will probably be useful in many circumstances.

Bugs 168953 and 212458 may be somewhat related - they're asking for different 
features, but the table selection tool should be able to help at least for the 
copy'n'paste (not so much for the highlighting).

I'll put together additional rationale and post it on the bug.


Jiri
-- 
Jiri Baum <jiri at baum.com.au>                   http://www.baum.com.au/sabik