Skip to content

Commit

Permalink
Fix IG bug related to auto/hard size
Browse files Browse the repository at this point in the history
When you have an autosizable field, IG would give focus to the hard size
input when you clicked on the field label.  This had the undesired
consequence that the hard sized value would be used in the simulation
even if the autosize radio button was still clicked.  This is bad and
the behavior is just weird.  The solution is as easy as removing the
correct two lines of code.

close #1476
  • Loading branch information
kbenne committed Jun 16, 2015
1 parent 86be6ee commit 6eefb16
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions openstudiocore/src/model_editor/InspectorGadget.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -800,9 +800,6 @@ void InspectorGadget::layoutText( QVBoxLayout* layout,
}
commentText->setObjectName("IDFcomment");

label->setFocusPolicy(Qt::ClickFocus);
label->setFocusProxy( text );

vbox->addWidget(commentText);

if(exists)
Expand Down

5 comments on commit 6eefb16

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2160 of 2213 tests passed)

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

Build Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (2191 of 2213 tests passed)

Build Badge Test Badge

Please sign in to comment.