diff options
| author | Justin Luth <jluth@mail.com> | 2023-11-06 16:22:26 -0500 |
|---|---|---|
| committer | Justin Luth <jluth@mail.com> | 2023-11-15 20:05:07 +0100 |
| commit | ac0b3ff84522a388e0d44b09bf681af13a8288b2 (patch) | |
| tree | 9a63854859e5dc31167323692968ef210f7901ea /editeng/source | |
| parent | 0fc96d9b54e5a64c528bbf7fcdf9b71d74be8920 (diff) | |
tdf#158031 editeng SID_REMOVE_HYPERLINK: use AlsoCheckBeforeCursor
This patch depends on prior patches for this bug report.
This is mostly just a clean-up patch to make it function like the others.
It was a lot like EDIT_HYPERLINK - because it selects the hyperlink first,
so it didn't need any fixing up.
However, there was one instance where KEYBOARD selection of
Remove Hyperlink in Draw could have been cancelled/slot-invalidated
if the mouse had been moved away from the field.
Change-Id: Id1b911f2548b8e2751ae34a2158ee4984dfc9a59
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159018
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
Diffstat (limited to 'editeng/source')
| -rw-r--r-- | editeng/source/misc/urlfieldhelper.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/editeng/source/misc/urlfieldhelper.cxx b/editeng/source/misc/urlfieldhelper.cxx index 1ed8647a15ae..d60d7be92934 100644 --- a/editeng/source/misc/urlfieldhelper.cxx +++ b/editeng/source/misc/urlfieldhelper.cxx @@ -16,7 +16,8 @@ void URLFieldHelper::RemoveURLField(EditView& pEditView) { pEditView.SelectFieldAtCursor(); - const SvxFieldData* pField = pEditView.GetFieldAtCursor(); + const SvxFieldItem* pFieldItem = pEditView.GetFieldAtSelection(); + const SvxFieldData* pField = pFieldItem ? pFieldItem->GetField() : nullptr; if (auto pUrlField = dynamic_cast<const SvxURLField*>(pField)) { ESelection aSel = pEditView.GetSelection(); |