iOS improvements: implement FS listener, improved touch behaviour, fix more file permission issues

Author: timothyschoen

Libraries/JUCE

@@ -2720,6 +2720,7 @@ bool Canvas::panningModifierDown() const
 #if JUCE_IOS
     return OSUtils::ScrollTracker::isScrolling();
 #endif
+    
     auto const& commandManager = editor->commandManager;
     // check the command manager for the keycode that is assigned to pan drag key
     auto const panDragKeycode = commandManager.getKeyMappings()->getKeyPressesAssignedToCommand(CommandIDs::PanDragKey).getFirst().getKeyCode();

Source/Canvas.cpp

@@ -596,7 +596,7 @@ class CanvasViewport final : public Viewport
     }
 
     void mouseWheelMove(MouseEvent const& e, MouseWheelDetails const& wheel) override
-    {
+    {        
         // Check event time to filter out duplicate events
         // This is a workaround for a bug in JUCE that can cause mouse events to be duplicated when an object has a MouseListener on its parent
         if (e.eventTime == lastScrollTime)

Source/CanvasViewport.h

@@ -18,16 +18,24 @@ class BouncingViewportAttachment final : public MouseListener
     }
 
     ~BouncingViewportAttachment() override = default;
+        
+#if JUCE_IOS
+    void mouseDrag(MouseEvent const& e) override
+    {
+        if(e.getDistanceFromDragStart() > 8) {
+            OSUtils::ScrollTracker::setAllowOneFingerScroll(true);
+        }
+    }
+#endif
 
     void mouseWheelMove(MouseEvent const& e, MouseWheelDetails const& wheel) override
     {
         if (!isBounceable)
             return;
-
         // Protect against receiving the same even twice
         if (e.eventTime == lastScrollTime)
             return;
-
+        
         lastScrollTime = e.eventTime;
 
         auto const deltaY = rescaleMouseWheelDistance(wheel.deltaY);

Source/Components/BouncingViewport.h

@@ -405,6 +405,9 @@ void PropertiesPanel::BoolComponent::mouseExit(MouseEvent const& e)
 
 void PropertiesPanel::BoolComponent::mouseUp(MouseEvent const& e)
 {
+    if(!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
+        return;
+    
     toggleStateValue.setValue(!getValue<bool>(toggleStateValue));
     repaint();
 }

Source/Components/PropertiesPanel.cpp

@@ -74,6 +74,9 @@ class WelcomePanel final : public Component
 
         void mouseUp(MouseEvent const& e) override
         {
+            if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
+                return;
+            
             if (clearButtonBounds.contains(e.getPosition())) {
                 auto const settingsTree = SettingsFile::getInstance()->getValueTree();
                 settingsTree.getChildWithName("RecentlyOpened").removeAllChildren(nullptr);
@@ -219,8 +222,8 @@ class WelcomePanel final : public Component
         {
             if (!getScreenBounds().reduced(12).contains(e.getScreenPosition()))
                 return;
-
-            if (!e.mods.isLeftButtonDown())
+            
+            if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
                 return;
 
             onClick();
@@ -633,7 +636,7 @@ class WelcomePanel final : public Component
 
         void mouseUp(MouseEvent const& e) override
         {
-            if (!e.mods.isLeftButtonDown())
+            if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
                 return;
 
             // If the cursor is no longer over the tile, don't trigger the tile

Source/Components/WelcomePanel.h

@@ -873,11 +873,16 @@ void Dialogs::showOpenDialog(std::function<void(URL)> const& callback, bool cons
     fileChooser->launchAsync(openChooserFlags,
         [callback, lastFileId](FileChooser const& fileChooser) {
             auto const result = fileChooser.getResult();
-
+            auto const resultURL = fileChooser.getURLResult();
+            
             auto const lastDir = result.isDirectory() ? result : result.getParentDirectory();
             if (result.exists()) {
+#if JUCE_IOS
+                // Create an input stream to access the security scoped resource
+                auto scopedAccessStream = resultURL.createInputStream(URL::InputStreamOptions(URL::ParameterHandling::inAddress));
+#endif
                 SettingsFile::getInstance()->setLastBrowserPathForId(lastFileId, lastDir);
-                callback(fileChooser.getURLResult());
+                callback(resultURL);
             }
             Dialogs::fileChooser = nullptr;
         });

Source/Dialogs/Dialogs.cpp

@@ -529,8 +529,11 @@ class PatchDisplay final : public Component {
         repaint();
     }
 
-    void mouseDown(MouseEvent const& e) override
+    void mouseUp(MouseEvent const& e) override
     {
+        if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
+            return;
+
         callback(info);
     }
 

Source/Dialogs/PatchStore.h

@@ -55,6 +55,9 @@ class ActionButton final : public Component {
 
     void mouseUp(MouseEvent const& e) override
     {
+        if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
+            return;
+        
         onClick();
     }
 

Source/Dialogs/PathsSettingsPanel.h

@@ -194,6 +194,9 @@ class SnapSettings final : public Component {
 
     void mouseUp(MouseEvent const& e) override
     {
+        if (!ModifierKeys::getCurrentModifiers().isLeftButtonDown())
+            return;
+        
         for (auto* group : buttonGroups) {
             group->dragToggledInteraction = false;
             group->repaint();