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

Move "Install App" box closer to top of the page

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22

      Description

      The "Install App" box on app pages currently is located in the middle-right part of an app's page. There is a lot of white space above it, which makes the page look unbalanced. Also, for many users, the most important functionality of the page is this very same "Install App" box.

      Can this box be moved closer to the top of the page?

      Please note that if this box can be moved, it would also need to move down a bit to make room for the "Edit" button when an author is logged in. However, this "Edit" button can probably be placed in another location, provided it's clear to an App author that they need to click this button (or link) in order to start editing the page.

        Attachments

        1. after-change.png
          after-change.png
          373 kB
        2. app-page.png
          app-page.png
          307 kB
        3. edit-page.png
          edit-page.png
          332 kB

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-2563 [ 18746 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - Submitted PR. Link: https://bitbucket.org/lorainelab/appstore/pull-requests/250
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          The branch is coming from a few commits back compared to where the master branch is pointing right now. Could you rebase on the latest commit of the master branch?

          Show
          ann.loraine Ann Loraine added a comment - - edited The branch is coming from a few commits back compared to where the master branch is pointing right now. Could you rebase on the latest commit of the master branch?
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          A (hopefully) quick request:

          • Could you attach 2 images, one showing the app page with the "Edit" button and another without it?

          The "Edit" button appears when the user is logged in and has write permission to the app. This could be an author of the app or an admin user.

          Show
          ann.loraine Ann Loraine added a comment - - edited A (hopefully) quick request: Could you attach 2 images, one showing the app page with the "Edit" button and another without it? The "Edit" button appears when the user is logged in and has write permission to the app. This could be an author of the app or an admin user.
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Attachment edit-page.png [ 15068 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Attachment app-page.png [ 15069 ]
          Hide
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - - edited

          Dr. [~aloraine], please find the images attached to the ticket. I also synched my branch with the master branch.

          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - - edited Dr. [~aloraine] , please find the images attached to the ticket. I also synched my branch with the master branch.
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Thank you! Please submit PR when convenient.

          Show
          ann.loraine Ann Loraine added a comment - Thank you! Please submit PR when convenient.
          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/250
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Chirag Chandrahas Shetty [ chirag24 ]
          ann.loraine Ann Loraine made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Assignee Chirag Chandrahas Shetty [ chirag24 ] Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Merged and deployed to https://devappstore5.bioviz.org. Ready for testing.

          Show
          ann.loraine Ann Loraine added a comment - - edited Merged and deployed to https://devappstore5.bioviz.org . Ready for testing.
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 7 Dec 14 - Dec 23 [ 110 ] Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8 [ 110, 111 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ssegu Sai Supreeth Segu (Inactive) added a comment -

          In app page,
          1. Space between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Resources" button.
          2. Space between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Error" tab.

          Also these changes are reflected in https://devappstore5.bioviz.org not in https://devappstore3.bioviz.org

          cc: [~aloraine] and Chirag Chandrahas Shetty

          Show
          ssegu Sai Supreeth Segu (Inactive) added a comment - In app page, 1. Space between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Resources" button. 2. Space between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Error" tab. Also these changes are reflected in https://devappstore5.bioviz.org not in https://devappstore3.bioviz.org cc: [~aloraine] and Chirag Chandrahas Shetty
          Hide
          ann.loraine Ann Loraine added a comment -

          Sorry about that - devappstore3 was not showing the correct branch. This will be fixed in the next minute or two when the playbooks finish running.
          attn: Sai Supreeth Segu

          Show
          ann.loraine Ann Loraine added a comment - Sorry about that - devappstore3 was not showing the correct branch. This will be fixed in the next minute or two when the playbooks finish running. attn: Sai Supreeth Segu
          chirag24 Chirag Chandrahas Shetty (Inactive) made changes -
          Attachment after-change.png [ 15087 ]
          Hide
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment -

          Adjusted gap between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Resources" button.
          PR: https://bitbucket.org/lorainelab/appstore/pull-requests/259/igbf-2708-adjust-distance-between-edit

          Show
          chirag24 Chirag Chandrahas Shetty (Inactive) added a comment - Adjusted gap between "Edit this page" and "Install app" is not consistent with space between "Install app" button and "Resources" button. PR: https://bitbucket.org/lorainelab/appstore/pull-requests/259/igbf-2708-adjust-distance-between-edit
          Hide
          ann.loraine Ann Loraine added a comment -

          Merged and deployed to devappstore3 and devappstore5. Both are now up-to-date with the latest master branch.

          Show
          ann.loraine Ann Loraine added a comment - Merged and deployed to devappstore3 and devappstore5. Both are now up-to-date with the latest master branch.
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Post-merge Testing In Progress [ 10003 ] Merged Needs Testing [ 10002 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Post-merge Testing In Progress [ 10003 ] Merged Needs Testing [ 10002 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8 [ 110, 111 ] Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22 [ 110, 111, 112 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          Hide
          ssegu Sai Supreeth Segu (Inactive) added a comment -

          Everything looks good. Closing the ticket

          Show
          ssegu Sai Supreeth Segu (Inactive) added a comment - Everything looks good. Closing the ticket
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Assignee Sai Supreeth Segu [ ssegu ] Chirag Chandrahas Shetty [ chirag24 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

            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: