Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19, Winter 5 Feb 22 - Mar 5, Winter 6 Mar 8 - Mar 19

      Description

      Note: this may now be redundant with IGBF-2723, a previous worked-on issue. Check on that before starting this.

      Here is a description of the problem:

      App pages contain two sections on the right side of the page that contain the app installation section (contains a button) and a section titled "RESOURCES" (all caps).

      The "install app" button is a core part of the functionality. Having it change depending on the widths of the screen creates a feeling of instability and fluidity, which sends a subtle negative message about interface reliability to the user.

      For this task, please investigate:

      • Can you modify the CSS so that the width of this column never increases or decreases? Ideally, it should not expand or contract but always have the same width. Please make sure to test with different widths and browsers.

      Note: If it seems too difficult, make a note of it here. Please only spend 1/2 day on it.

        Attachments

          Issue Links

            Activity

            Hide
            ann.loraine Ann Loraine added a comment -

            Chirag Chandrahas Shetty - Can you take a look?

            Show
            ann.loraine Ann Loraine added a comment - Chirag Chandrahas Shetty - Can you take a look?
            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - diff: https://bitbucket.org/chirag06/appstore/commits/71754f94fed95012d4a278f3507e705d30f4c08d
            Hide
            ann.loraine Ann Loraine added a comment -

            Chirag Chandrahas Shetty - please provide well-named images showing:

            • current behavior of app page at the relevant screen widths
            • new behavior of your branch at the relevant screen widths

            Relevant screen widths (or others as you see fit)

            • wide - 1650 pixels
            • small - 500 pixels

            Example of well-named image files:

            • 1650-current-[git hash].png
            • 1650-branch-[git hash].png

            Note that the file name contains image width in pixels, a string indicated whether the image is from the existing code or the new code, and a git hash that allows us to easily identify the tip of the branch.

            Show
            ann.loraine Ann Loraine added a comment - Chirag Chandrahas Shetty - please provide well-named images showing: current behavior of app page at the relevant screen widths new behavior of your branch at the relevant screen widths Relevant screen widths (or others as you see fit) wide - 1650 pixels small - 500 pixels Example of well-named image files: 1650-current- [git hash] .png 1650-branch- [git hash] .png Note that the file name contains image width in pixels, a string indicated whether the image is from the existing code or the new code, and a git hash that allows us to easily identify the tip of the branch.
            Hide
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

            [~aloraine] I have attached the screenshots as per your request. Please have a look at them.

            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - [~aloraine] I have attached the screenshots as per your request. Please have a look at them.
            Hide
            ann.loraine Ann Loraine added a comment -

            This looks great. Please submit PR when ready.

            Show
            ann.loraine Ann Loraine added a comment - This looks great. Please submit PR when ready.
            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/268/igbf-2728-make-app-page-right-col-of-fix
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged but not yet deployed to testing site.

            Show
            ann.loraine Ann Loraine added a comment - Merged but not yet deployed to testing site.
            Hide
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

            Testing: Please test the changes by changing the browser size in Safari, chrome, Firefox and Edge.

            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - Testing: Please test the changes by changing the browser size in Safari, chrome, Firefox and Edge.
            Hide
            ann.loraine Ann Loraine added a comment -

            Deployed to https://devappstore3.bioviz.org

            I'm noticing something a bit weird is happening to the "RESOURCES" block when the page width decreases. The little icons turn a pale shade of gray and also the spacing between the various listed items changes.

            Chirag Chandrahas Shetty - could you look into it?

            Show
            ann.loraine Ann Loraine added a comment - Deployed to https://devappstore3.bioviz.org I'm noticing something a bit weird is happening to the "RESOURCES" block when the page width decreases. The little icons turn a pale shade of gray and also the spacing between the various listed items changes. Chirag Chandrahas Shetty - could you look into it?
            Hide
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - - edited

            That is happening because of the bootstrap's inbuilt nav class that we are using for the "RESOURCE" block. I cannot find the exact code where it is happening but we can add some code to copy some of the properties that we want and use that class instead of nav.
            cc:[~aloraine]

            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - - edited That is happening because of the bootstrap's inbuilt nav class that we are using for the "RESOURCE" block. I cannot find the exact code where it is happening but we can add some code to copy some of the properties that we want and use that class instead of nav. cc: [~aloraine]
            Hide
            ann.loraine Ann Loraine added a comment -

            Question for Chirag Chandrahas Shetty:

            Why can't you find the exact code where it is happening? I am not sure what is meant in the previous comment.

            Show
            ann.loraine Ann Loraine added a comment - Question for Chirag Chandrahas Shetty : Why can't you find the exact code where it is happening? I am not sure what is meant in the previous comment.
            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - diff: https://bitbucket.org/chirag06/appstore/commits/2dc1ee1cceb2ddc64e0d6d1f7fdbeb2ee12114ae
            Hide
            ann.loraine Ann Loraine added a comment -

            Please submit PR when convenient.

            Show
            ann.loraine Ann Loraine added a comment - Please submit PR when convenient.
            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/272/igbf-2728-fix-app-page-resources-block
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Merged and deployed on https://appstore3.bioviz.org

            Show
            ann.loraine Ann Loraine added a comment - - edited Merged and deployed on https://appstore3.bioviz.org
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment -

            Tested on https://devappstore5.bioviz.org/. The "install app" button maintains its width across screen sizes as intended.

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - Tested on https://devappstore5.bioviz.org/ . The "install app" button maintains its width across screen sizes as intended.

              People

              • Assignee:
                chirag24 Chirag Chandrahas Shetty (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: