omz:forum

    • Register
    • Login
    • Search
    • Recent
    • Popular

    Welcome!

    This is the community forum for my apps Pythonista and Editorial.

    For individual support questions, you can also send an email. If you have a very short question or just want to say hello — I'm @olemoritz on Twitter.


    CaptureMedia

    Pythonista
    6
    22
    19861
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • ccc
      ccc last edited by

      @Tony, Thanks for the explanations on the desire for extension points and log_message(). Makes sense to me. I posted a pull request to re-add log_message() and will try to put the extension points back into the GitHub source in the next 24 hours (if someone does not do so before I can).

      Can we come up with a different name for did_load()? Ole had reserved the did_load() name for views that load ui resources from a .pyui file. A did_load() in a script that does not use .pyui might confuse the casual reader. Given that all ui elements are created at runtime without .pyui, we should use a similar yet different name.

      1 Reply Last reply Reply Quote 0
      • ?
        A Former User last edited by

        @ccc Good. Maybe... consider though... should the consumer of a class need to know or bother whether behind the scenes it's ui was implemented with the ui editor or with handwritten code?? The use of did_load here was deliberate to create consistency and transparency.

        1 Reply Last reply Reply Quote 0
        • ccc
          ccc last edited by

          I submitted a pull request which I hope solves the issues that you mentioned above.

          I also opened a new issue on cancel and close.

          1 Reply Last reply Reply Quote 0
          • ?
            A Former User last edited by

            @ccc Well done. Yes, it's a good point about detecting the cancel. The evaluate JavaScript returns immediately so that doesn't help. There's no POST when the user cancels... so again that doesn't help. It's hard to wait for an absence of the file when there's no knowing how long a video record might last.

            In the base class... after cancel, tap outside the popover to close it and then the server closes because of the will close.

            The sub class close button is the same scenario as the close from the POST I think... it seems to be a quirk of a programmatic close of a view presented in a popover that will_close doesn't automatically get called on the view in that case. Hence the second line at the end of POST to fix that. (present in a panel doesn't have that quirk I think, nor probably the other presents though I haven't tested.)

            The sub class was a quick knock up to show the idea, but more properly I think the button item action would need to invoke both will close and close because of the quirk.

            1 Reply Last reply Reply Quote 0
            • ?
              A Former User last edited by

              @ccc It's a bit of a fudge, but this addition to the base class seems to fix the quirk of present in a popover programmatic close for both the POST and the sub class button item action (and the fix line from POST can be removed)...

              def close(self):
              		self.will_close()
              		super(CaptureMedia, self).close()
              
              1 Reply Last reply Reply Quote 0
              • ?
                A Former User last edited by

                Updated source to version 2.0

                1 Reply Last reply Reply Quote 0
                • techteej
                  techteej last edited by

                  @tony Just updated.

                  1 Reply Last reply Reply Quote 0
                  • ?
                    A Former User last edited by

                    @techteej Thanks.

                    p.s. I'm working on adding in a settings button and my SettingsSheet class for a switch to turn warnings on/off.

                    1 Reply Last reply Reply Quote 0
                    • techteej
                      techteej last edited by

                      @tony I have to test this on an iOS 7 iPad, which I don't see very often (it's a relatives) so good luck! Hopefully @ccc will help you. I can do what I can when I get the iOS 7 iPad.

                      1 Reply Last reply Reply Quote 0
                      • ?
                        A Former User last edited by

                        @techteej Thanks :) Likewise when iOS 8 is out I'll have a way to try to get it working on that.

                        1 Reply Last reply Reply Quote 0
                        • jmv38
                          jmv38 last edited by

                          capture_media generates an error on line 115 on my ipad air ios7.... any fix?

                          1 Reply Last reply Reply Quote 0
                          • JonB
                            JonB last edited by

                            Looks like you should replace self._f with os.path.split(__file__)[1].
                            You might also want to try one of the older versions (github history) since it seems like people checked in work without testing.

                            1 Reply Last reply Reply Quote 0
                            • jmv38
                              jmv38 last edited by

                              @JonB thanks.
                              i've replaced the self._f at 2 locations, but there are still errors.
                              Which version in the history would your recommend?

                              1 Reply Last reply Reply Quote 0
                              • Webmaster4o
                                Webmaster4o last edited by

                                Where is a working link?

                                1 Reply Last reply Reply Quote 0
                                • First post
                                  Last post
                                Powered by NodeBB Forums | Contributors