Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336497 - [quick diff] 'Press F2 for focus' hides 1 line of code
Summary: [quick diff] 'Press F2 for focus' hides 1 line of code
Status: RESOLVED WORKSFORME
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-07 07:31 EST by Deepak Azad CLA
Modified: 2011-02-07 08:56 EST (History)
2 users (show)

See Also:

Attachments
screenshot (15.66 KB, image/png)
2011-02-07 07:31 EST, Deepak Azad CLA 		no flags Details
screenshot of javadoc hover (103.91 KB, image/png)
2011-02-07 08:47 EST, Deepak Azad CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Deepak Azad CLA 2011-02-07 07:31:30 EST 
Created attachment 188431 [details]
screenshot

Quick diff is a nice way to see the original code, the hover also shows the code nicely with its original indentation so that it is extremely easy to read the code. 

However, 'Press F2 for focus' hides 1 line of the code (see screenshot). This is a bit irritating, and reduces the usability of quick diff hover at times.
Comment 1 Dani Megert CLA 2011-02-07 08:20:56 EST 
>Created attachment 188431 [details]
>screenshot
Do you have an example/steps?
Comment 2 Markus Keller CLA 2011-02-07 08:34:07 EST 
The screenshot is actually OK. I think Deepak just wants to hide the affordance.

=> Uncheck "Show affordance in hover on how to make it sticky" on the "Text Editors" preference page.
Comment 3 Deepak Azad CLA 2011-02-07 08:47:05 EST 
Created attachment 188435 [details]
screenshot of javadoc hover

(In reply to comment #2)
> The screenshot is actually OK. I think Deepak just wants to hide the
> affordance.
> 
> => Uncheck "Show affordance in hover on how to make it sticky" on the "Text
> Editors" preference page.
Ha, this is exactly what I wanted, though only for quick diff hover :)

While I do not particularly mind doing this for all hovers, in at least one case the javadoc hover looks a bit odd with the preference disabled. (See screenshot)
Comment 4 Markus Keller CLA 2011-02-07 08:52:42 EST 
> Ha, this is exactly what I wanted, though only for quick diff hover :)

I don't want yet another preference, so you have to live with all or nothing ;-)

> While I do not particularly mind doing this for all hovers, in at least one
> case the javadoc hover looks a bit odd with the preference disabled. (See
> screenshot)

That's a temporary glitch right after toggling the preference. Resolves itself as soon as you close/reopen the editor.
Comment 5 Deepak Azad CLA 2011-02-07 08:56:36 EST 
(In reply to comment #4)
> > Ha, this is exactly what I wanted, though only for quick diff hover :)
> 
> I don't want yet another preference, so you have to live with all or nothing
> ;-)
> 
> > While I do not particularly mind doing this for all hovers, in at least one
> > case the javadoc hover looks a bit odd with the preference disabled. (See
> > screenshot)
> 
> That's a temporary glitch right after toggling the preference. Resolves itself
> as soon as you close/reopen the editor.
Yeah, it works after I reopen the editor. I guess I can live with all or nothing. Thanks.