Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      The app boxes (tiles) are great, but their horizontal dimension (width) and text spacing seems a bit wonky.

      If I make the page bigger, the tiles expand and the text spreads out to fill the space. This seems fine to me.

      However, if I make the page smaller, then tiles get smaller and the text does not look very nice - it touches the thin line outlining the box, and the space between authors in the author list expands.

      This makes it a bit difficult for App developers to ensure that their tile will always look nice no matter what. It would be good to make the tiles have a more stable and predictable look, no matter how the page is being resized. Also, it might give the users greater confidence in the site if key visual elements (such as the tiles) have the same appearance regardless of the window size.

        Attachments

          Issue Links

            Activity

            Hide
            sameer Sameer Shanbhag (Inactive) added a comment -

            Hi Professor [~aloraine],

            Please Review the following Pull Request:

            Change Log:

            • Deal with the Common Header Placement
            • Remove 'App With Curated Category' Hyper Link from App Page
            • Align App Tiles from the homepage to work with all Screen Sizes
            • Small CSS Changes
            Show
            sameer Sameer Shanbhag (Inactive) added a comment - Hi Professor [~aloraine] , Please Review the following Pull Request: https://bitbucket.org/lorainelab/appstore/pull-requests/243/igbf-2692-improve-header-app-tiles-and Change Log: Deal with the Common Header Placement Remove 'App With Curated Category' Hyper Link from App Page Align App Tiles from the homepage to work with all Screen Sizes Small CSS Changes
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged and deployed to https://devappstore3.biovi.org.
            Please note that https://devappstore5.bioviz.org deploys the previous commit.

            Show
            ann.loraine Ann Loraine added a comment - Merged and deployed to https://devappstore3.biovi.org . Please note that https://devappstore5.bioviz.org deploys the previous commit.
            Hide
            ann.loraine Ann Loraine added a comment -

            Author names text has a lot of extra whitespace between different lines. It does not affect functionality, but looks a bit weird.
            I made a new ticket to address this: https://jira.transvar.org/browse/IGBF-2705

            Show
            ann.loraine Ann Loraine added a comment - Author names text has a lot of extra whitespace between different lines. It does not affect functionality, but looks a bit weird. I made a new ticket to address this: https://jira.transvar.org/browse/IGBF-2705
            Hide
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

            [~aloraine] and Sameer Shanbhag: I think that the app description needs to have padding on its right. Please refer the image above.

            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - [~aloraine] and Sameer Shanbhag : I think that the app description needs to have padding on its right. Please refer the image above.
            Hide
            ann.loraine Ann Loraine added a comment -

            FYI: Latest master branch is now also deployed to https://devappstore5.bioviz.org.

            Show
            ann.loraine Ann Loraine added a comment - FYI: Latest master branch is now also deployed to https://devappstore5.bioviz.org .
            Hide
            ann.loraine Ann Loraine added a comment -

            Regarding comment from Sai Supreeth Segu: Do please make sure you are not accidentally using the older css. (I am not seeing the same error.)

            Sometimes these files get cached. You may need to view the site using an "incognito" window or a different browser.

            Show
            ann.loraine Ann Loraine added a comment - Regarding comment from Sai Supreeth Segu : Do please make sure you are not accidentally using the older css. (I am not seeing the same error.) Sometimes these files get cached. You may need to view the site using an "incognito" window or a different browser.
            Hide
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

            Sorry I cleared my cache and it looks fine.

            Show
            chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - Sorry I cleared my cache and it looks fine.
            Hide
            ann.loraine Ann Loraine added a comment -

            Also please note there is a new ticket to fix the lower margin - see: IGBF-2715 "Fix lower margin of app tile"

            Show
            ann.loraine Ann Loraine added a comment - Also please note there is a new ticket to fix the lower margin - see: IGBF-2715 "Fix lower margin of app tile"
            Hide
            ssegu Sai Supreeth Segu (Inactive) added a comment -

            Everything looks good. Closing this Ticket.

            Show
            ssegu Sai Supreeth Segu (Inactive) added a comment - Everything looks good. Closing this Ticket.

              People

              • Assignee:
                sameer Sameer Shanbhag (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: