• Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Register
  • Login
UBports Robot Logo UBports Forum
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Search
  • Register
  • Login

Testing the upgrade to Qt 5.12

Scheduled Pinned Locked Moved OS
137 Posts 19 Posters 50.0k Views 8 Watching
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.
    • D Offline
      dobey @kugiigi
      last edited by dobey 11 Apr 2020, 15:15 4 Nov 2020, 15:13

      @kugiigi The problem is that Image {} can't magically determine the intent of app devs showing splash screen, and some define it incorrectly. There's an updated splash in gallery-app on GitLab but it hasn't been released yet. But even these new splashes will have some problems in certain situations, and won't work for all possible window sizes under Lomiri. It just happens that apparently some changes in Qt 5.12 are exposing these issues to us.

      So, we need to define the expected behavior for X-Ubuntu-Splash-Image better, and stick to that, and then if there are still more complaints about size or scaling, the apps need to be fixed. So, what is the opinion of @CiberSheep and UX group here?

      The old Ubuntu platform guide says this on it:

      File name of an image present in the directory stated in the Path entry. The splash image is centered on the splash screen and displayed in its actual size (ie, it's not stretched or shrunk and aspect ratio is kept).

      M C 2 Replies Last reply 4 Nov 2020, 15:17 Reply Quote 0
      • M Offline
        mymike @dobey
        last edited by 4 Nov 2020, 15:17

        @dobey said in Testing the upgrade to Qt 5.12:

        File name of an image present in the directory stated in the Path entry. The splash image is centered on the splash screen and displayed in its actual size (ie, it's not stretched or shrunk and aspect ratio is kept).

        For vector images, the splash screen has always scaled the image to fill the minimum value between window width and height. Not sure if we want to change that or keep it...

        1 Reply Last reply Reply Quote 0
        • D Offline
          dobey
          last edited by 4 Nov 2020, 16:54

          OK. Did a little more poking at the Splash, and I think I have it working more like it was before, without the weird artifacts happening. Please upgrade and test again. Thanks!

          1 Reply Last reply Reply Quote 0
          • D Offline
            dobey
            last edited by 4 Nov 2020, 20:42

            Well, that didn't seem to really fix everything, and with more poking about at the X-Ubuntu-Splash-Image handling I'm surprised it every worked as described in the platform guide. I can't seem to get a configuration of the Image {} component which provides the behavior described there, and which also provides a usable experience with the sample set of apps installed on my test device, nor in my VM running unity8 on 16.04.

            1 Reply Last reply Reply Quote 0
            • D Offline
              danfro
              last edited by 4 Nov 2020, 21:25

              Just pulled the latest devel update for my E5, devel.

              Calendar app does crash on startup. http://paste.ubuntu.com/p/F4WjzkkQh9/

              Notes app does open, but too only gives blank screen when opening a date picker. http://paste.ubuntu.com/p/XYrxkX2PNd/

              FluffyChat works, also bottom edge. Although when opeing that the first time it took a few seconds to open up.

              Teleports + dekko blank screen on startup.

              uTorch does crash after it has lost focus.

              Farkle, sprint app, ThemeSwitch, Logviewer, terminal, music app, gallery app, weather app, docviewer, file manager, calculator are working (basic functions).

              Ok, thats for a first feedback.

              D 1 Reply Last reply 4 Nov 2020, 21:55 Reply Quote 0
              • C Offline
                CiberSheep @dobey
                last edited by 4 Nov 2020, 21:35

                @dobey said in Testing the upgrade to Qt 5.12:

                The old Ubuntu platform guide says this on it:

                File name of an image present in the directory stated in the Path entry. The splash image is centered on the splash screen and displayed in its actual size (ie, it's not stretched or shrunk and aspect ratio is kept).

                I think this is a good way of going. On Qt 5.9 the image is scaled to the smallest size but that works kind of funny on the tablet.

                If that's possible to implement, we can add that to the HIG and change any app that need fixing afterwards.

                Another planet, another time, another universe!

                D 1 Reply Last reply 4 Nov 2020, 22:03 Reply Quote 0
                • D Offline
                  dobey @danfro
                  last edited by 4 Nov 2020, 21:55

                  @danfro said in Testing the upgrade to Qt 5.12:

                  Calendar app does crash on startup

                  Is the pastebin from the log file, or from using the run-click.sh script I posted above to run it from CLI?

                  Notes app does open, but too only gives blank screen when opening a date picker.

                  I've opened a PR against notes app to fix this now.

                  uTorch does crash after it has lost focus.

                  It seems to work OK here on my e4.5. Can you run it using the script I posted above to see what happens?

                  D 1 Reply Last reply 6 Nov 2020, 16:40 Reply Quote 0
                  • D Offline
                    dobey @CiberSheep
                    last edited by 4 Nov 2020, 22:03

                    @CiberSheep I don't think it can be implemented as described, and give a good result. IIRC, the idea was for it to allow people to provide a phone-sized image, but that clearly doesn't translate to tablets or PCs.

                    I can implement it such that there is a maximum square area that matches the smallest edge, and we can have the image scaled/cropped for that area, but there will still be situations where the result it gives us will not be what app devs desire for their splash images.

                    C 1 Reply Last reply 4 Nov 2020, 22:39 Reply Quote 0
                    • C Offline
                      CiberSheep @dobey
                      last edited by 4 Nov 2020, 22:39

                      @dobey said in Testing the upgrade to Qt 5.12:

                      I can implement it such that there is a maximum square area that matches the smallest edge, and we can have the image scaled/cropped for that area, but there will still be situations where the result it gives us will not be what app devs desire for their splash images.

                      • It has to be a square, then?
                      • I would say scaled rather than cropped if possible
                      • Developer, we will have to work around the limitations. All is knowing how it works before hand.

                      Another planet, another time, another universe!

                      D 1 Reply Last reply 5 Nov 2020, 14:50 Reply Quote 0
                      • D Offline
                        dobey @CiberSheep
                        last edited by 5 Nov 2020, 14:50

                        @CiberSheep said in Testing the upgrade to Qt 5.12:

                        It has to be a square, then?

                        I would say yes, because it's the only way we can have any sort of guarantee for something reasonable across all form factors.

                        I would say scaled rather than cropped if possible

                        I think both PreserveAspectCrop and PreserveAspectFit will likely behave the same way with what I described, anyway, since the image will always be smaller than the window.

                        Developer, we will have to work around the limitations. All is knowing how it works before hand.

                        Right. We need a proper and accurate description of the behavior, for which we can judge against when any complaint is made about that behavior to determine if said complaint is a valid issue or not (ie, whether we need to fix it in Lomiri, or app dev needs to fix their image). I don't think the current description is accurate for what the code is doing, and if we modify the code to actually behave that way, it will not give the desired effect in most cases. Though, if we want the existing description to stand, I can certainly make it work that way.

                        C 1 Reply Last reply 5 Nov 2020, 23:22 Reply Quote 0
                        • C Offline
                          CiberSheep @dobey
                          last edited by 5 Nov 2020, 23:22

                          @dobey said in Testing the upgrade to Qt 5.12:

                          I would say yes, because it's the only way we can have any sort of guarantee for something reasonable across all form factors.

                          Vertical splash images from Sailfish or Seedsheep look ok only on portrait, true. Square it is then

                          Though, if we want the existing description to stand, I can certainly make it work that way.

                          The only thing I can think of is: scale down the splash image to the size of the screen but when scaling up, maximum to the size of the image.
                          Say, fit a 512x512 image in smaller screen but show the corresponding image size in tablet (without scaling it up)

                          The idea would be showing a relatively same image across devices.

                          Another planet, another time, another universe!

                          D 1 Reply Last reply 6 Nov 2020, 14:57 Reply Quote 0
                          • D Offline
                            dobey @CiberSheep
                            last edited by 6 Nov 2020, 14:57

                            @CiberSheep OK. So, in the current build I have changed the code to accurately behave according to the existing description. In this way there is no scaling, and the image is centered.

                            Please try it with many apps and let me know what you think.

                            C 1 Reply Last reply 8 Nov 2020, 00:40 Reply Quote 1
                            • K Offline
                              kugiigi
                              last edited by 6 Nov 2020, 15:28

                              A bit less important but I'll report anyway and with the upcoming Volla and Fxtec, this might be more noticeable to users 🙂

                              The icons in the indicator gets squished in the external display with the exemption of some.
                              screenshot20201105_232325591.jpg

                              And going back to the device screen expands them. I think I've encountered this issue before from to time to time though so this isn't exactly new but with Qt5.12, it seems that it always happen.
                              screenshot20201105_233140947.png

                              1 Reply Last reply Reply Quote 0
                              • D Offline
                                danfro @dobey
                                last edited by 6 Nov 2020, 16:40

                                @dobey Ok, finally found time to run some tests.

                                Is the pastebin from the log file, or from using the run-click.sh script I posted above to run it from CLI?

                                It is from logviewer.

                                I've opened a PR against notes app to fix this now.

                                Great! 👍

                                It seems to work OK here on my e4.5. Can you run it using the script I posted above to see what happens?

                                Ok, whenever I try to start an app via terminal app with bash runclick.sh com.ubuntu.clock, I do get the same error:

                                aa-exec_ ERROR: profile 'com.ubuntu.clock_clock_3.12.5' does not exist

                                1 Reply Last reply Reply Quote -1
                                • D Offline
                                  danfro
                                  last edited by 6 Nov 2020, 19:29

                                  Just my findings, may they help others:

                                  Running Rodneys script from terminal app from within the phone does not seem to work. But using adb shell works great!

                                  If you need to find the proper names of the apps to run them, use click list (thanks Rodney for this hint). Copy/paste possible in adb shell for even more convenience. 😉

                                  1 Reply Last reply Reply Quote 0
                                  • C Offline
                                    CiberSheep @dobey
                                    last edited by 8 Nov 2020, 00:40

                                    @dobey said in Testing the upgrade to Qt 5.12:

                                    @CiberSheep OK. So, in the current build I have changed the code to accurately behave according to the existing description. In this way there is no scaling, and the image is centered.

                                    Please try it with many apps and let me know what you think.

                                    Updated and testing.

                                    • I think it works good for staged splash
                                    • With side staged or windowed, not so. Is it possible to scale down to fit the window size on those cases (when splash image is bigger than the window)?

                                    31079bc1-ddd9-452b-a8b5-15268f1bd88e-imatge.png

                                    Another planet, another time, another universe!

                                    1 Reply Last reply Reply Quote 0
                                    • D Offline
                                      danfro
                                      last edited by danfro 11 Aug 2020, 10:48 8 Nov 2020, 10:47

                                      Hm. When changing from one view to another in calendar app, the first time I enter a view, the header gets messed up. After switching back and entering a second time, things look fine.

                                      Just me? (Testing a few other things, so might be.)

                                      screenshot20201108_113117418.png

                                      The only entry in log (Logviewer) that seems to be new is this:

                                      file:///opt/click.ubuntu.com/com.ubuntu.calendar/0.8.4.2/qml/EventListModel.qml:183: TypeError: Value is undefined and could not be converted to an object
                                      file:///opt/click.ubuntu.com/com.ubuntu.calendar/0.8.4.2/qml/DayView.qml:94: TypeError: Cannot call method 'scrollToTime' of null
                                      
                                      C D 2 Replies Last reply 8 Nov 2020, 11:56 Reply Quote 0
                                      • C Offline
                                        CiberSheep @danfro
                                        last edited by 8 Nov 2020, 11:56

                                        @danfro this might be a state that has no default or the when condition is checked before the boolean is ready

                                        Another planet, another time, another universe!

                                        1 Reply Last reply Reply Quote 0
                                        • D Offline
                                          dobey @danfro
                                          last edited by 9 Nov 2020, 15:29

                                          @danfro said in Testing the upgrade to Qt 5.12:

                                          Just me? (Testing a few other things, so might be.)

                                          Nope, but I haven't found anything in logs that would point to the cause, other than the fact that Tabs is deprecated.

                                          1 Reply Last reply Reply Quote 0
                                          • D Offline
                                            danfro
                                            last edited by 21 Nov 2020, 06:18

                                            In notes app, when opening a note and going back to notes list, the header disappears. I think notes and calendar need to be stripped of the deprecated components to work properly with Qt 5.12.

                                            Header of notes app is the bad boy I suppose. Tabs the bad boy for calendar.

                                            D 1 Reply Last reply 8 Dec 2020, 19:35 Reply Quote 0
                                            • First post
                                              Last post