Re: The screwdriver is back!

Petr Hrebejk <[email protected]>
Newsgroups gmane.comp.java.netbeans.user-interface
Message-ID <[email protected]>
Cau,

I don't know what is more ugly, screwdriver or the green ones. Maybe HIE will 
come up with something nicer.

I agree with having the hint, but I disagree that it would help to get rid of 
the icons.

1) If there is not the @Override in the then there would a warning but the 
warning icon does not show what warning that is.

2) Clicking the icon gets you to the super method. Clicking a hint or @Overrides 
annotation obviously doesn't. Go To super implementation is less discoverable IMO.

3) Id the navigation does not work please file a bug.

Hrebejk

Jesse Glick wrote:
> For a while we had overrides/implements icons which were green "O"s and 
> "I"s, which was easy enough to remember, if not very elegant. Now we are 
> back to a cleaned-up version of the 5.x icons.
> 
> http://deadlock.nbextras.org/fisheye/changelog/netbeans/java/editor?cs=MAIN:phrebejk:20070221161628 
> 
> 
> Why the screwdriver, or whatever it is?
> 
> For that matter, why do we need two different icons? IMHO we could just 
> use the "overrides" icon. All I usually care about is knowing that this 
> method's signature is constrained by some supertype, i.e. that the 
> @Override annotation could be used. The details are available in the 
> tooltip if you care anyway.
> 
> More radical idea: get rid of the override/implements annotations 
> completely for 1.5+ sources. Instead, add an editor hint to add 
> @Override to any method which could use it but does not currently have 
> it. (Jackpot has a transformer like this too.) I would actually find 
> this more valuable - @Override helps prevent stupid mistakes when 
> editing supertypes (just as C#'s language keyword does), and once your 
> code is fully @Override-annotated, you won't need to see the distracting 
> editor glyph icons any more. "Go to Super Implementation" would then 
> take care of the navigation aspect of the current annotations. (Assuming 
> it actually worked - currently it doesn't.)

> -J.
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.