anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-12-24 13:48:59 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix bug 8124. We enable LFUN_INSET_MODIFY in InsetBox if it

Browse Source 
is intended for us, i.e., if the first argument is "box". Maybe we
should also check to make sure that the rest of the arguments are
good, but we used just to return true no matter what the argument
was, so this is an improvement.

LFUN_INSET_MODIFY is really kind of a mess.
This commit is contained in:
Richard Heck 2012-04-14 17:32:25 -04:00
parent 3a53a52b1c
commit 264948348e
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/insets/InsetBox.cpp
View File

@ -211,14 +211,20 @@ bool InsetBox::getStatus(Cursor & cur, FuncRequest const & cmd,
{
switch (cmd.action()) {
case LFUN_INSET_MODIFY:
if (cmd.getArg(0) == "changetype") {
case LFUN_INSET_MODIFY: {
string const first_arg = cmd.getArg(0);
if (first_arg == "changetype") {
string const type = cmd.getArg(1);
flag.setOnOff(type == params_.type);
flag.setEnabled(!params_.inner_box || type != "Framed");
return true;
}
if (first_arg == "box") {
flag.setEnabled(true);
return true;
}
return InsetCollapsable::getStatus(cur, cmd, flag);
}
case LFUN_INSET_DIALOG_UPDATE:
flag.setEnabled(true);

3
status.20x
View File

@ -122,6 +122,9 @@ What's new
- Allow the <Enter> key to add a new branch in document settings.
Only <Ctrl+Enter> and numpad-<Enter> will also close the dialog.
- Box settings dialog didn't work if you were actually inside the box
(bug 8124).
- Fix tooltip for "Use Non-TeX Fonts" (bug 7787).
- Fix pasting of LATIN CAPITAL LETTER SHARP S (bug 8057).