Question about Bug 402110

[Simone Gaiarin]

Hi Steven,
great to hear you want to contribute to Okular.

I suggest you the method I use for locating the relevant areas of code for
a particular feature/bug:
- Install ripgrep
- From the top level okular folder run `rg RELEVANT_SEARCH_TERM`, for
example `rg TableSel` given that you are looking for something related to
table selection or `rg Table` (a bit vague though).
- This leads us to ui/pageview.cpp (the file where we have more matches in
this case)
- Open the file and search for 'TableSel' and have a look at the code in
those areas to figure out what it does
- At this point try to search for something relevant in those areas, for
example 'clipboard' or 'copy'
- At line 2866 there is the instruction `QClipboard *cb =
QApplication::clipboard();` inside a switch case
`Okular::Settings::EnumMouseMode::TableSelect` inside a method
`PageView::mouseReleaseEvent(QMouseEvent *e)`. This seems the relevant area.
- Now put a breakpoint somewhere here and run the code, try to reproduce
the bug and see if it does indeed happen here

I hope this helps.

If other developers have better methods to locate relevant areas of code,
please share it.

Note:
You can achieve the same without ripgrep and using `grep -Ir
RELEVANT_SEARCH_TERM`, it's just slower.

On Sat, Dec 5, 2020 at 10:42 PM stevengkc714 <stevengkc714 at protonmail.com>
wrote:

> Hello. I'm new to contributing to KDE and I want to work on this issue in
> Okular: https://bugs.kde.org/show_bug.cgi?id=402110. The issue mentions
> changing the table select tool so that it only copies to the clipboard
> after pressing Ctrl+C. Where in the code would be a good place to implement
> this? Thanks in advance.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20201206/374b71d1/attachment.htm>