Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 313914

Summary:

[Webapp] Incorrect use of Hand pointer

Product:

[Eclipse Project] Platform

Reporter:

Deepak Azad <deepakazad>

Component:

User Assistance

Assignee:

Chris Goldthorpe <cgold>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

cgold, ChrisAustin

Version:

3.6

Target Milestone:

3.7 M6

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Patch	none

Description Deepak Azad

2010-05-21 10:12:55 EDT

The buttons (Highlight Search Terms, Go Back, Go forward, Print etc) on the top right in the Help Window use a hand pointer, which is incorrect as a hand pointer indicates a hyperlink which most of these buttons are not. Maybe you could use hand pointer for 'Go back' and 'Go Forward' buttons but similar buttons (buttons on the toolbar in Java perspective) elsewhere in the Eclipse IDE use the arrow pointer.

Comment 1 Deepak Azad

2010-05-21 10:13:19 EDT

Build id: I20100519-1548

Comment 2 Deepak Azad

2010-09-12 08:08:20 EDT

ping

Comment 3 Chris Goldthorpe

2010-09-20 11:00:17 EDT

I saw this bug when it came in but I haven't looked into it yet, it does not seem severe so I am working on some other bugs which have higher priority first.

Comment 4 Chris Goldthorpe

2011-02-03 18:38:13 EST

Chris A, do you have an opinion on this? It seems that we have been using a hand for a long time and I don't really notice any more. It's an easy enough change to make.

Comment 5 Chris Austin

2011-02-04 09:28:16 EST

(In reply to comment #4)
> Chris A, do you have an opinion on this? It seems that we have been using a
> hand for a long time and I don't really notice any more. It's an easy enough
> change to make.

Yeah - I personally don't have a strong opinion, I think I kind of like the hand pointers, but it wouldn't bother me if they went away.

Comment 6 Chris Goldthorpe

2011-02-04 14:15:59 EST

Created attachment 188355 [details]
Patch

The buttons already indicate their selectability by getting a border which makes them appear to pop out when they are hovered over so I agree that it would be more consistent with the workbench to use the default cursor. This patch also makes the maximize/minimize button behave like the other buttons.

Comment 7 Chris Goldthorpe

2011-02-04 14:16:47 EST

Patch committed to HEAD, Fixed