Bug 546995 - [code mining] Render multiple LineHeaders for same line below each other
Summary: [code mining] Render multiple LineHeaders for same line below each other
Status: CLOSED DUPLICATE of bug 544773
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 4.12   Edit
Hardware: All All
: P3 enhancement with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-05-06 03:53 EDT by Niko Stotz CLA
Modified: 2019-05-06 09:17 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Niko Stotz CLA 2019-05-06 03:53:37 EDT
Currently, if there are multiple LineHeaderCodeMinings on the same line, they are rendered next to each other (i.e. horizontally), separated by " | ".

Example:

1    class Hello {
2        int i;
         Code Mining 1 | Code Mining 2 | Code Mining 3
3        boolean b = this.equals(1);
...

Specificially for Bug 540443, rendering them below each other (i.e. vertically) would make more sense.

Example:

1    class Hello {
2        int i;
         Code Mining 1
         Code Mining 2
         Code Mining 3
3        boolean b = this.equals(1);
...

In general, I'd argue LineHeaderCodeMinings might tend to be quite long, so they would be easier to read in vertical rendering.

We could also introduce something like VerticalLineHeaderCodeMining in order not to change existing behavior. Mixing them with regular LineHeaderCodeMining might be difficult, though.
Comment 1 Sebastian Zarnekow CLA 2019-05-06 09:17:03 EDT

*** This bug has been marked as a duplicate of bug 544773 ***