Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2138

Improve box-release-info on apps page

    Details

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

      Description

      Improve box-release-info on apps pages to make it clearer what user needs to do to install and run Apps.

      See: https://www.dropbox.com/s/7k8ue6ha3l1ggc5/apppage.pptx?dl=0

      Task Summary:

      • No matter if IGB is running, always show text indicating platform compatibility:

      Works with IGB [release.platform_compatibility]

      See:

      https://www.dropbox.com/s/7k8ue6ha3l1ggc5/apppage.pptx?dl=0

      • Remove hyperlink from platform_compatibility string
      • Instead of "Version: [release.Bundle_version]" display "App Version: [release.Bundle_version]"
      • Replace "IGB" text above the button with "To install and run this App, start IGB."
      • Button label: Replace "Get IGB" with "Get the latest IGB" when IGB is not running

        Attachments

          Issue Links

            Activity

            Hide
            sameer Sameer Shanbhag (Inactive) added a comment -

            Code Changes Pushed:
            https://bitbucket.org/sshanbh1/sameer-appstorefork/commits/7a3f4c98611e0a18cc5d6bf2ece886b3fafd82df

            Please check and let me know if any other changes are required.

            Show
            sameer Sameer Shanbhag (Inactive) added a comment - Code Changes Pushed: https://bitbucket.org/sshanbh1/sameer-appstorefork/commits/7a3f4c98611e0a18cc5d6bf2ece886b3fafd82df Please check and let me know if any other changes are required.
            Hide
            ann.loraine Ann Loraine added a comment -

            There's a lot of changed code so I'm having some trouble predicting how the behavior is different. But I'm OK with merging this in. If you get the chance to submit a PR today, please do. Sameer Shanbhag

            Show
            ann.loraine Ann Loraine added a comment - There's a lot of changed code so I'm having some trouble predicting how the behavior is different. But I'm OK with merging this in. If you get the chance to submit a PR today, please do. Sameer Shanbhag
            Hide
            sameer Sameer Shanbhag (Inactive) added a comment -

            Earlier the Page was divided into 4 sections Top Right - Top Left - Bottom Right - Bottom Left I converted it into 2 sections Right and Left Respectively so the majority of the code change is just "copy and paste" on the Appstore App Page.

            I will make a Pull Request so that Riddhi Jagdish Patil can use this as a baseline to do the other task in her To-Do's

            Pull Request Can be Found here:
            https://bitbucket.org/lorainelab/appstore/pull-requests/157/igbf-2138-improve-editor-panel/diff

            Show
            sameer Sameer Shanbhag (Inactive) added a comment - Earlier the Page was divided into 4 sections Top Right - Top Left - Bottom Right - Bottom Left I converted it into 2 sections Right and Left Respectively so the majority of the code change is just "copy and paste" on the Appstore App Page. I will make a Pull Request so that Riddhi Jagdish Patil can use this as a baseline to do the other task in her To-Do's Pull Request Can be Found here: https://bitbucket.org/lorainelab/appstore/pull-requests/157/igbf-2138-improve-editor-panel/diff
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged but not yet deployed to dev-appstore-2

            Show
            ann.loraine Ann Loraine added a comment - Merged but not yet deployed to dev-appstore-2
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Deployed on dev-appstore-2.
            Reviewed and changes needed:

            Case 1: IGB is not running:

            1) Fix punctuation and text to match design:

            Change:

            "To install and run this app, Start IGB"

            to

            "To install and run this app, start IGB!"

            2) Fix layout of button text within button

            • Center horizontally and vertically
            • Inject more space between download icon and text
            • Download icon needs to be white not dark blue (currently is not visible)

            3) Change button text.

            • Change from: "Get Latest IGB"
            • Change to: "Get latest IGB"

            4) Change "works with" text:

            • Remove bold font for IGB platform_compatibility string.
            • Remove hyperlink surrounding above.
            • Remove bold face for above.
            • Keep "Works with IGB" bold face
            • Remove colon - : character

            5) When user hovers mouse over the button, make sure that the browser displays the link destination URL in the usual place, e.g., lower left corner in Chrome.

            6) App Version is not being shown. This text should never NOT be shown, regardless of whether IGB is running or not.

            Show
            ann.loraine Ann Loraine added a comment - - edited Deployed on dev-appstore-2. Reviewed and changes needed: Case 1 : IGB is not running: 1) Fix punctuation and text to match design: Change: "To install and run this app, Start IGB" to "To install and run this app, start IGB!" 2) Fix layout of button text within button Center horizontally and vertically Inject more space between download icon and text Download icon needs to be white not dark blue (currently is not visible) 3) Change button text. Change from: "Get Latest IGB" Change to: "Get latest IGB" 4) Change "works with" text: Remove bold font for IGB platform_compatibility string. Remove hyperlink surrounding above. Remove bold face for above. Keep "Works with IGB" bold face Remove colon - : character 5) When user hovers mouse over the button, make sure that the browser displays the link destination URL in the usual place, e.g., lower left corner in Chrome. 6) App Version is not being shown. This text should never NOT be shown, regardless of whether IGB is running or not.
            Hide
            sameer Sameer Shanbhag (Inactive) added a comment - - edited
            Show
            sameer Sameer Shanbhag (Inactive) added a comment - - edited Pull Request Raised with the requested changes: https://bitbucket.org/lorainelab/appstore/pull-requests/157/igbf-2138-improve-editor-panel/diff
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged not yet deployed to dev-appstore-2

            Show
            ann.loraine Ann Loraine added a comment - Merged not yet deployed to dev-appstore-2
            Hide
            ann.loraine Ann Loraine added a comment -

            Deployed to dev-appstore-2

            Show
            ann.loraine Ann Loraine added a comment - Deployed to dev-appstore-2
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Removing database because "works with" logic was changed. Database may need to be refreshed.

            Show
            ann.loraine Ann Loraine added a comment - - edited Removing database because "works with" logic was changed. Database may need to be refreshed.
            Hide
            pbole Pawan Bole (Inactive) added a comment -

            While testing the ticket I found a couple of issues and I have created new tasks for the same:

            1) Browser to show link destination URL on hover of the 'Get latest IGB' button.
            https://jira.transvar.org/browse/IGBF-2153
            2) Format for the version of IGB that apps support.
            https://jira.transvar.org/browse/IGBF-2154

            Other test cases mentioned are working as expected, hence marking the task as complete.

            Show
            pbole Pawan Bole (Inactive) added a comment - While testing the ticket I found a couple of issues and I have created new tasks for the same: 1) Browser to show link destination URL on hover of the 'Get latest IGB' button. https://jira.transvar.org/browse/IGBF-2153 2) Format for the version of IGB that apps support. https://jira.transvar.org/browse/IGBF-2154 Other test cases mentioned are working as expected, hence marking the task as complete.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: