I have long argued for this feature and it's great that you started to implement it as an option.
Yet, I have to say that currently the implementation is half-a.. err. -hearted - most importantly:
If the target row is not at all currently visible on the screen then please simply center it.
If the editor has to scroll anyway then it's a good time to show as much context as possible.
This is especially silly if the target file itself wasn't open before. E.g. do "Go to Symbol: setVisible" and choose java.awt.Component (assuming that class isn't already showing in any editor tab). Idea opens the class, but places the caret row with the method at the very bottom of the window. What good is that?
Could you please fix that before Demetra goes gold?!