path: root/include
diff options
authorTor Lillqvist <>2020-03-18 10:58:52 +0200
committerTor Lillqvist <>2020-03-20 15:11:27 +0100
commitf4cb7a249ebe62c7009bf0ca2f02ea89c1a75a68 (patch)
treed17c52673b70c4858ca173384c5427315f7fdd6a /include
parent5ccb7ecf7cda92934d63cc171674b6c4f7f51af7 (diff)
Clarify the WIP-ness of setMobilePhone() and setTablet() a bit
Change-Id: Iee8b773e23ebd3fbc1ba5bf2c56d644d0844cb2d Reviewed-on: Tested-by: Jenkins CollaboraOffice <> Reviewed-by: Tor Lillqvist <> Reviewed-on: Tested-by: Jenkins
Diffstat (limited to 'include')
1 files changed, 5 insertions, 0 deletions
diff --git a/include/comphelper/lok.hxx b/include/comphelper/lok.hxx
index d88ddfd1dfa5..43191db90ec0 100644
--- a/include/comphelper/lok.hxx
+++ b/include/comphelper/lok.hxx
@@ -29,6 +29,11 @@ namespace LibreOfficeKit
COMPHELPER_DLLPUBLIC void setActive(bool bActive = true);
+// Note that currently it is undefined behaviour to call both setMobilePhone() and setTablet(). This
+// will be remedied in the future. For now, just make sure you call just either for a view, and just
+// once, with the bool parameter as true. In the future, this will probably be changed into using an
+// enum for the kind of the view, that can be DESKTOP, MOBILEPHONE, or TABLET.
// Tell that LOK view is on a mobile phone (regardless what its pixel resolution is, whether its form factor is "phablet" or not)
COMPHELPER_DLLPUBLIC void setMobilePhone(int nViewId, bool bIsMobilePhone);