diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-07-12 15:43:46 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-07-12 20:54:15 +0200 |
| commit | 0b05a04f978688aedce90e259a31873de82a2d2a (patch) | |
| tree | c825911468c0647cace4a2a266f832ff8be757ba /dbaccess | |
| parent | bcea211b66910ee6cf6a51e4baa950b3f897e310 (diff) | |
clarify what WantMouseEvent is for
and add tristate machine support that we also need
WantMouseEvent is if the first click in a browse cell that activates it should
be passed to the control that appears on click as if it was itself clicked on
primarily for the the CheckBoxController to toggle it immediately.
Rework to explicitly toggle the checkbox in that implementation rather than
rely on passing fake mouse click/release/tracking events.
seeing as the main dbaccess table design view doesn't auto-launch its listboxes
on clicking in a cell, but the sub create index dialog does, drop the auto-launch
in the index dialog to match the main table design rathern than add a feature
to weld::ComboBox to auto-launch the popup for that one solitary case
Change-Id: Ie3d3f2ecf55d3d5b0b02b85ca09a6ca64bb800e9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/98603
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'dbaccess')
| -rw-r--r-- | dbaccess/source/ui/dlg/indexfieldscontrol.cxx | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx index e0613e02ac17..dd959a05dda8 100644 --- a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx +++ b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx @@ -53,7 +53,6 @@ constexpr auto BROWSER_STANDARD_FLAGS = BrowserMode::COLUMNSELECTION | BrowserMo void SetAdditionalModifyHdl(const Link<DbaMouseDownListBoxController&,void>& _rHdl); protected: - virtual bool WantMouseEvent() const override { return true; } virtual void callModifyHdl() override; }; |