<html>
<head>
<base href="https://bugs.documentfoundation.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Accessibility: No keyboard shortcut to open a HYPERLINK in Calc cells"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=123159#c17">Comment # 17</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Accessibility: No keyboard shortcut to open a HYPERLINK in Calc cells"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=123159">bug 123159</a>
from <span class="vcard"><a class="email" href="mailto:erack@redhat.com" title="Eike Rathke <erack@redhat.com>"> <span class="fn">Eike Rathke</span></a>
</span></b>
<pre>Implementation detail..
(In reply to Heiko Tietze from <a href="show_bug.cgi?id=123159#c10">comment #10</a>)
<span class="quote">> The function ScGlobal::OpenURL() is called in ScGridWindow::MouseButtonUp().
> Cleanest solution would be to separate the part that takes various
> situations into account. But this is some effort since it's closely linked
> to the mouse position.</span >
The mouse position there is only needed to obtain the URL from the cell via
ScGridWindow::GetEditUrl() and for the tiled rendering case. Separating that
out and factoring out all the address conversion then would only need a
GetEditUrl() on the currently active cell for the keyboard case.
For ScGlobal::OpenURL() it's about the active view shell/frame and mouse click
modifier keys. The view shell/frame is the same for a Ctrl+Enter (or whatever)
and whether to evaluate the mouse click modifier (and what happens if not)
could be made a parameter, defaulted to current behaviour.
<span class="quote">> Alternatively, we might simulate a mouse click in the
> UNO command that needs to be added for the keyboard interaction.</span >
How should that work? You'd have to navigate the mouse to position it on cells
to emulate the click. Sounds more like a pain than a gain.
<span class="quote">> Or
> duplicate the code maybe with limited capabilities (no VbaEvents, no mobile
> device.</span >
That does not sound like a good idea..</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>