Bug 541317 - [code mining] Gives the capability to update code mining for a given line
Summary: [code mining] Gives the capability to update code mining for a given line
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 4.8   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-19 09:55 EST by Angelo ZERR CLA
Modified: 2018-11-19 09:59 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Angelo ZERR CLA 2018-11-19 09:55:39 EST
In JDT CodeMining, a user reported the following issue https://github.com/angelozerr/jdt-codemining/issues/52

The idea is to show mining only for the selected line. This feature requires a new API for codemining to compute and refresh the UI just for a given line.

Today I have not worked on this issue, but I think it could be a cool feature.
Comment 1 Mickael Istria CLA 2018-11-19 09:59:18 EST
I also like this idea!

> to compute and refresh the UI just for a given line.

I don't think we need to compute it for a single line. Because if you think about the use-case here, the user is most likely going to navigate code and read multiple lines on the same file. So having them all computed can be useful.
It seems to me that a strategy to only "filter" for the current line in the CodeMiningManager would be enough. The good thing is that it wouldn't require extensions to add anything to support this; it'd be up to consumer (usually the editor) to deal with enablement of the code mining (current line vs visible lines).