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

Improve behavior and appearance of download pages when user agent unrecognized user-agent

    Details

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

      Description

      If a user running an unrecognized browser accesses IGB installer download pages, the "download" button does not appear.

      On the Downloads page (for the released version of IGB), a slightly cryptic link appears "Other ways to download IGB" in more or less the spot the button would be.

      On the "Early Access page" (for main-JDK8 branch latest installer build), a link appears that reads "Click here for the released and tested version of IGB."

      It would be better if some other behavior could be developed that would be less confusing to users.

      For testing, see IGBF-3385 comment for how to use Chrome Developer Tools to change your user-agent and observe the above behavior.

        Attachments

          Issue Links

            Activity

            ann.loraine Ann Loraine created issue -
            ann.loraine Ann Loraine made changes -
            Field Original Value New Value
            Epic Link IGBF-1765 [ 17855 ]
            ann.loraine Ann Loraine made changes -
            Link This issue relates to IGBF-3388 [ IGBF-3388 ]
            ann.loraine Ann Loraine made changes -
            Labels Beginner
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            Hide
            ann.loraine Ann Loraine added a comment -

            Attn: Nowlan Freese

            Adding this to the backlog with label "Beginner" for new joiners to work on.

            Show
            ann.loraine Ann Loraine added a comment - Attn: Nowlan Freese Adding this to the backlog with label "Beginner" for new joiners to work on.
            Hide
            nfreese Nowlan Freese added a comment -

            Currently, we determine the user agent and then remove the d-none class (which hides the button) for that button. If the user agent doesn't match mac/windows/linux then all of the buttons remain hidden. We could add a fourth button/class so that if the user agent isn't mac/windows/linux we display some kind of message that we do not support their current operating system, or we only support mac/windows/linux.

            Show
            nfreese Nowlan Freese added a comment - Currently, we determine the user agent and then remove the d-none class (which hides the button) for that button. If the user agent doesn't match mac/windows/linux then all of the buttons remain hidden. We could add a fourth button/class so that if the user agent isn't mac/windows/linux we display some kind of message that we do not support their current operating system, or we only support mac/windows/linux.
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ]
            pkulzer Paige Kulzer made changes -
            Sprint Fall 6 [ 229 ]
            pkulzer Paige Kulzer made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            pkulzer Paige Kulzer added a comment -

            I've added a warning message to both the Downloads and Early Downloads pages that triggers when the user agent is unrecognized. See attached for an example of how that looks.

            Branch: https://bitbucket.org/pkulzer-lorainelab/bioviz/branch/IGBF-3397

            Show
            pkulzer Paige Kulzer added a comment - I've added a warning message to both the Downloads and Early Downloads pages that triggers when the user agent is unrecognized. See attached for an example of how that looks. Branch: https://bitbucket.org/pkulzer-lorainelab/bioviz/branch/IGBF-3397
            pkulzer Paige Kulzer made changes -
            Attachment warning-unrecognized-useragent.png [ 18954 ]
            pkulzer Paige Kulzer made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ] Nowlan Freese [ nfreese ]
            ann.loraine Ann Loraine made changes -
            Sprint Fall 6 [ 229 ] Fall 6, Fall 7 [ 229, 230 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            nfreese Nowlan Freese made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            Hide
            pkulzer Paige Kulzer added a comment -

            Nowlan Freese noticed an issue during testing where the new warning message popped up when the Windows installer was being displayed. I've fixed this issue, and I've also made the warning message itself more clear with a link to the other downloads page.

            Show
            pkulzer Paige Kulzer added a comment - Nowlan Freese noticed an issue during testing where the new warning message popped up when the Windows installer was being displayed. I've fixed this issue, and I've also made the warning message itself more clear with a link to the other downloads page.
            Hide
            nfreese Nowlan Freese added a comment -

            An additional issue is what the expected behavior should be if the user agent is unknown or not supported on the early access downloads page. Showing nothing or a warning may be confusing to a user as no options would be present. I'm also imagining a scenario where a user on mac/windows/linux is using a browser that hides there user agent. They would see no option to download an early access installer. I think we should by default show all three installer options for the early access page. The assumption is that if a user chooses the early access IGB installer they are advanced enough to know what operating system they should choose.

            Show
            nfreese Nowlan Freese added a comment - An additional issue is what the expected behavior should be if the user agent is unknown or not supported on the early access downloads page. Showing nothing or a warning may be confusing to a user as no options would be present. I'm also imagining a scenario where a user on mac/windows/linux is using a browser that hides there user agent. They would see no option to download an early access installer. I think we should by default show all three installer options for the early access page. The assumption is that if a user chooses the early access IGB installer they are advanced enough to know what operating system they should choose.
            Hide
            pkulzer Paige Kulzer added a comment -

            I've updated the commit to forgo a warning on the early access installer page in favor of showing all three installer options by default.

            Ready for review!

            Show
            pkulzer Paige Kulzer added a comment - I've updated the commit to forgo a warning on the early access installer page in favor of showing all three installer options by default. Ready for review!
            Hide
            nfreese Nowlan Freese added a comment -

            Code looks good and the download and early access pages are working as expected.

            Ready for PR

            Show
            nfreese Nowlan Freese added a comment - Code looks good and the download and early access pages are working as expected. Ready for PR
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ] Paige Kulzer [ pkulzer ]
            nfreese Nowlan Freese made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            Show
            pkulzer Paige Kulzer added a comment - PR: https://bitbucket.org/lorainelab/bioviz/pull-requests/187
            pkulzer Paige Kulzer made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ] Ann Loraine [ aloraine ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            PR is merged. New code is deployed to bioviz.org. Ready for testing.

            Show
            ann.loraine Ann Loraine added a comment - - edited PR is merged. New code is deployed to bioviz.org. Ready for testing.
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            nfreese Nowlan Freese made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ]
            Hide
            nfreese Nowlan Freese added a comment -

            Tested on Chrome.

            Downloads page shows the correct installer for the user agent. If user agent is unknown or not supported then it shows a notification that installers for Mac, Windows, and Linux are available and it includes a link to the otherDownloads page.

            Early Access page shows the three installer buttons for Mac, Windows, Linux.

            All buttons appeared to be working and downloaded installers.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Tested on Chrome. Downloads page shows the correct installer for the user agent. If user agent is unknown or not supported then it shows a notification that installers for Mac, Windows, and Linux are available and it includes a link to the otherDownloads page. Early Access page shows the three installer buttons for Mac, Windows, Linux. All buttons appeared to be working and downloaded installers. Closing ticket.
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ] Paige Kulzer [ pkulzer ]
            nfreese Nowlan Freese made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

              People

              • Assignee:
                pkulzer Paige Kulzer
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: