android/sdremote/res/values/strings.xml | 12 ++-- android/sdremote/src/org/libreoffice/impressremote/PresentationActivity.java | 2 sd/source/ui/view/drviews7.cxx | 26 ++++++---- 3 files changed, 25 insertions(+), 15 deletions(-)
New commits: commit dcbb11df33fdb32be58694907cfe633e39103204 Author: Michael Meeks <michael.me...@suse.com> Date: Wed Feb 13 14:35:40 2013 +0000 sdremote: hide TCP/IP server selection dialog menu item Now shown only in experimental mode (ie. when TCP/IP is enabled) to reduce user-confusion. Said dialog is not needed for bluetooth. (cherry picked from commit 13cb088a2d8902822e3eb5607ba4cac81ad4d2f2) Signed-off-by: Thorsten Behrens <tbehr...@suse.com> diff --git a/sd/source/ui/view/drviews7.cxx b/sd/source/ui/view/drviews7.cxx index f2d69be..f077d30 100644 --- a/sd/source/ui/view/drviews7.cxx +++ b/sd/source/ui/view/drviews7.cxx @@ -26,6 +26,7 @@ #include <editeng/eeitem.hxx> #include <editeng/flditem.hxx> #include <editeng/outlobj.hxx> +#include <officecfg/Office/Common.hxx> #include <officecfg/Office/Impress.hxx> #include <svx/svxids.hrc> #include <svx/svdpagv.hxx> @@ -51,7 +52,6 @@ #include <comphelper/processfactory.hxx> #include <sfx2/request.hxx> - #include <svx/pfiledlg.hxx> #include <svx/grafctrl.hxx> #include <svtools/cliplistener.hxx> @@ -996,15 +996,25 @@ void DrawViewShell::GetMenuState( SfxItemSet &rSet ) rSet.DisableItem(SID_ZOOM_PREV); } - bool bDisableSdremoteForGood = false; + if( SFX_ITEM_AVAILABLE == rSet.GetItemState( SID_REMOTE_DLG ) ) + { + + bool bDisableSdremoteForGood = false; #ifndef ENABLE_SDREMOTE - bDisableSdremoteForGood = true; + bDisableSdremoteForGood = true; #endif - uno::Reference< uno::XComponentContext > xContext = comphelper::getProcessComponentContext(); - if ( bDisableSdremoteForGood || ( xContext.is() && !officecfg::Office::Impress::Misc::Start::EnableSdremote::get( xContext ) ) ) - { - rSet.ClearItem(SID_REMOTE_DLG); - rSet.DisableItem(SID_REMOTE_DLG); + uno::Reference< uno::XComponentContext > xContext = comphelper::getProcessComponentContext(); + if ( xContext.is() ) + bDisableSdremoteForGood |= ! ( officecfg::Office::Common::Misc::ExperimentalMode::get( xContext ) && + + officecfg::Office::Impress::Misc::Start::EnableSdremote::get( xContext ) ); + + // This dialog is only useful for TCP/IP remote control + // which is unusual, under-tested and a security issue. + if ( bDisableSdremoteForGood ) + { + rSet.Put(SfxVisibilityItem(SID_REMOTE_DLG, false)); + } } // EditText aktiv commit e412e2750a1821e31099e6defc5d62eace061281 Author: Michael Meeks <michael.me...@suse.com> Date: Wed Feb 13 13:51:08 2013 +0000 sdremote: avoid crash when fiddling with elapsed time. (cherry picked from commit 03b941e9ae63626a3336aabdb6a3485835ec6f9d) Signed-off-by: Thorsten Behrens <tbehr...@suse.com> diff --git a/android/sdremote/src/org/libreoffice/impressremote/PresentationActivity.java b/android/sdremote/src/org/libreoffice/impressremote/PresentationActivity.java index 0de6b32..d3a0352 100644 --- a/android/sdremote/src/org/libreoffice/impressremote/PresentationActivity.java +++ b/android/sdremote/src/org/libreoffice/impressremote/PresentationActivity.java @@ -485,7 +485,7 @@ public class PresentationActivity extends SherlockFragmentActivity { @Override public boolean onEditorAction(TextView tv, int aID, KeyEvent aEvent) { - if (aEvent.getKeyCode() == KeyEvent.KEYCODE_ENTER) { + if (aEvent != null && aEvent.getKeyCode() == KeyEvent.KEYCODE_ENTER) { long aTime = 0; try { commit eb72920b5a9c5e7a46bb65281bbd0cc41fbba12e Author: Tor Lillqvist <t...@iki.fi> Date: Wed Feb 13 09:26:42 2013 +0200 Ah, manually adding computers is for non-Bluetooth ones So yeah, need to keep this feature. Spell Wi-Fi correctly, and mention Wi-Fi in the menu button. Change-Id: I4cd6f0689a6ff9647f355d4b1404be84ec0c0138 (cherry picked from commit ea264f0c8165eb0c62bf624af5f450b20898469f) Signed-off-by: Thorsten Behrens <tbehr...@suse.com> diff --git a/android/sdremote/res/values/strings.xml b/android/sdremote/res/values/strings.xml index 80493f9..a08edd1 100644 --- a/android/sdremote/res/values/strings.xml +++ b/android/sdremote/res/values/strings.xml @@ -23,7 +23,7 @@ <string name="options_switchcomputer">Switch computer</string> <string name="blankscreen_return">Return to Slide</string> <string name="bluetooth">Bluetooth</string> - <string name="wifi">WI-FI</string> + <string name="wifi">Wi-Fi</string> <string name="selector_noservers">Searching for computersâ¦</string> <string name="selector_delete">Remove computer</string> <string name="selector_choose_a_computer">Choose a Computer</string> @@ -42,7 +42,7 @@ <string name="about_copyright">Copyright © 2012 LibreOffice Contributors and/or their affiliates.</string> <string name="about_licence">This app is released under the Mozilla Public License, v. 2.0.</string> <string name="about_libraries">This app uses android-coverflow\n\tCopyright © 2011, Polidea\n\tNew BSD License.\n\nThis app uses ActionBarSherlock:\n\tCopyright 2012 Jake Wharton\n\tLicensed under the Apache License, Version 2.0 (the "License")</string> - <string name="addserver">Add Computer Manually</string> + <string name="addserver">Add Wi-Fi Computer Manually</string> <string name="addserver_entername">Computer name:</string> <string name="addserver_enteraddress">Computer IP address or hostname:</string> <string name="addserver_remember">Remember this computer next time</string> commit 9f1dd0e8481c166b6c16e87548a8adf6b8b28d7c Author: Tor Lillqvist <t...@iki.fi> Date: Wed Feb 13 09:17:53 2013 +0200 Use consistent terminology Why "Choose a Computer" but "Add Server"? Let's always call them "computers" in the UI. And in fact I think the feature to add a computer manually is kinda pointless; is it too much to ask people to have their machines discoverable? I doubt keeping a computer not discoverable through Bluetooth thwarts a determined attacker from finding it anyway. (I am not saying that just finding it would mean compromising it.) Change-Id: Ib8041a8a793d84bee24587adac582c5fad03d1be (cherry picked from commit 056968fae7fa598e8cb528fb6b45c999de33a102) Signed-off-by: Thorsten Behrens <tbehr...@suse.com> diff --git a/android/sdremote/res/values/strings.xml b/android/sdremote/res/values/strings.xml index e84b7fa..80493f9 100644 --- a/android/sdremote/res/values/strings.xml +++ b/android/sdremote/res/values/strings.xml @@ -25,7 +25,7 @@ <string name="bluetooth">Bluetooth</string> <string name="wifi">WI-FI</string> <string name="selector_noservers">Searching for computersâ¦</string> - <string name="selector_delete">Remove server</string> + <string name="selector_delete">Remove computer</string> <string name="selector_choose_a_computer">Choose a Computer</string> <string name="selector_dialog_connecting">Attempting to connect to {0}â¦</string> <string name="selector_dialog_connectionfailed">Impress Remote could not connect to {0}.</string> @@ -42,10 +42,10 @@ <string name="about_copyright">Copyright © 2012 LibreOffice Contributors and/or their affiliates.</string> <string name="about_licence">This app is released under the Mozilla Public License, v. 2.0.</string> <string name="about_libraries">This app uses android-coverflow\n\tCopyright © 2011, Polidea\n\tNew BSD License.\n\nThis app uses ActionBarSherlock:\n\tCopyright 2012 Jake Wharton\n\tLicensed under the Apache License, Version 2.0 (the "License")</string> - <string name="addserver">Add Server</string> - <string name="addserver_entername">Server name:</string> - <string name="addserver_enteraddress">Server address as IP or hostname:</string> - <string name="addserver_remember">Remember this server next time</string> + <string name="addserver">Add Computer Manually</string> + <string name="addserver_entername">Computer name:</string> + <string name="addserver_enteraddress">Computer IP address or hostname:</string> + <string name="addserver_remember">Remember this computer next time</string> <string name="addserver_add">Add</string> <string name="addserver_cancel">Cancel</string> <string name="reconnect_description1">Your connection has been dropped.</string>
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits