mirror of
https://git.lyx.org/repos/lyx.git
synced 2024-12-25 14:04:25 +00:00
Disable LFUN_INSET_DISSOLVE in tabulars
When in a tabular cell, "this" is just a lone InsetText, while cur.inset() is the whole tabular. This makes a big difference, especially when one wants to count cells. Fixes bug #9954. (cherry picked from commit 4caf167dbd24722421ae8d94a2c103bf41a7667d)
This commit is contained in:
parent
7444b894f8
commit
67f77a07f0
@ -293,7 +293,8 @@ void InsetText::doDispatch(Cursor & cur, FuncRequest & cmd)
|
||||
bool const main_inset = &buffer().inset() == this;
|
||||
bool const target_inset = cmd.argument().empty()
|
||||
|| cmd.getArg(0) == insetName(lyxCode());
|
||||
bool const one_cell = nargs() == 1;
|
||||
// cur.inset() is the tabular when this is a single cell (bug #9954)
|
||||
bool const one_cell = cur.inset().nargs() == 1;
|
||||
|
||||
if (!main_inset && target_inset && one_cell) {
|
||||
// Text::dissolveInset assumes that the cursor
|
||||
@ -325,7 +326,8 @@ bool InsetText::getStatus(Cursor & cur, FuncRequest const & cmd,
|
||||
bool const main_inset = &buffer().inset() == this;
|
||||
bool const target_inset = cmd.argument().empty()
|
||||
|| cmd.getArg(0) == insetName(lyxCode());
|
||||
bool const one_cell = nargs() == 1;
|
||||
// cur.inset() is the tabular when this is a single cell (bug #9954)
|
||||
bool const one_cell = cur.inset().nargs() == 1;
|
||||
|
||||
if (target_inset)
|
||||
status.setEnabled(!main_inset && one_cell);
|
||||
|
@ -71,7 +71,10 @@ What's new
|
||||
|
||||
- Raise GuiView on buffer switch to bring it topmost in multi-window mode.
|
||||
|
||||
- Fix cursor position after Redo of a document settings change (bug 10097).
|
||||
- Fix cursor position after Redo of a document settings change (bug
|
||||
10097).
|
||||
|
||||
- Disable Edit>Dissolve Inset in tabulars (bug 9954).
|
||||
|
||||
- Shortcut preferences: ask the user for removing bindings when using the
|
||||
"restore" button (#9174).
|
||||
|
Loading…
Reference in New Issue
Block a user