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

Detect Safari on Genome Dashboard

    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:
      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

      Due to security concerns, the Safari browser won't allow a Web page loaded via https to open URLs with http (non-secured) protocol. Therefore, we need to always provide some kind of warning to Safari users about this when using Web pages that try to contact IGB via http.

      For this task, implement some kind of improvement to the Genome Dashboard application (production link: https://www.bioviz.org/genome-dashboard) so that if a user visits using Safari, they will understand why the application is not working and what to do about it.

      Be sure to fully read the background information about this – see linked issues. Also, search for every Jira ticket containing the word "Safari" in the description to ensure you understand the issue fully and how we have dealt with it in the past.

        Attachments

          Issue Links

            Activity

            Hide
            cdias1 Chester Dias (Inactive) added a comment -

            Banner appears for all browsers with exception of Chromium-based browsers, Firefox, and Edge.

            Show
            cdias1 Chester Dias (Inactive) added a comment - Banner appears for all browsers with exception of Chromium-based browsers, Firefox, and Edge.
            Hide
            cdias1 Chester Dias (Inactive) added a comment - - edited

            The view functionality too will work only for Chromium-based browsers, Firefox, and Edge.
            The messages displayed were finalized based on the discussion in today's meeting with Nowlan Freese and Chirag Chandrahas Shetty.

            [~aloraine] Please let me know your thoughts on this so I can make the final changes if any.
            I have tried the new banner in Google Chrome, Edge, Firefox, and Safari and it works as expected.

            Show
            cdias1 Chester Dias (Inactive) added a comment - - edited The view functionality too will work only for Chromium-based browsers, Firefox, and Edge. The messages displayed were finalized based on the discussion in today's meeting with Nowlan Freese and Chirag Chandrahas Shetty . [~aloraine] Please let me know your thoughts on this so I can make the final changes if any. I have tried the new banner in Google Chrome, Edge, Firefox, and Safari and it works as expected.
            Show
            cdias1 Chester Dias (Inactive) added a comment - Please review: https://bitbucket.org/chesterdias/genome-dashboard-local-chester/branch/IGBF-2731#diff
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Please make a note of what you learned from the feedback and how (or if) you made any changes to the interface.

            Show
            ann.loraine Ann Loraine added a comment - - edited Please make a note of what you learned from the feedback and how (or if) you made any changes to the interface.
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            The message "unsupported browser type" and "Redirection..." is probably too technical. A user who has never before used this interface will likely not understand what "redirection" means.

            Show
            ann.loraine Ann Loraine added a comment - - edited The message "unsupported browser type" and "Redirection..." is probably too technical. A user who has never before used this interface will likely not understand what "redirection" means.
            Hide
            ann.loraine Ann Loraine added a comment -

            Please see attached image showing the popup that appears when a user clicks a species tile and IGB is not running.

            Can you match the look & feel for the popup that appears if a user tries to click a tile when running Safari browser? Note how the message is friendly, brief, informal, and addresses the user directly (using second person singular.)

            Show
            ann.loraine Ann Loraine added a comment - Please see attached image showing the popup that appears when a user clicks a species tile and IGB is not running. Can you match the look & feel for the popup that appears if a user tries to click a tile when running Safari browser? Note how the message is friendly, brief, informal, and addresses the user directly (using second person singular.)
            Hide
            ann.loraine Ann Loraine added a comment -

            Chester Dias - when ready, can you give a demo?

            Show
            ann.loraine Ann Loraine added a comment - Chester Dias - when ready, can you give a demo?
            Hide
            ann.loraine Ann Loraine added a comment -

            Change requests:

            • Change yellow banner message to "To open a genome in IGB, use a different Web browser: Firefox, Chrome, or Edge."
            • Change popup message as follows:
            • heading - "Unsupported Web browser"
            • message - "To open a genome in IGB, use a different Web browser: Firefox, Chrome, or Edge."
            • add button: "Close" (make it look the same as in start-igb-messagepng")
            Show
            ann.loraine Ann Loraine added a comment - Change requests: Change yellow banner message to "To open a genome in IGB, use a different Web browser: Firefox, Chrome, or Edge." Change popup message as follows: heading - "Unsupported Web browser" message - "To open a genome in IGB, use a different Web browser: Firefox, Chrome, or Edge." add button: "Close" (make it look the same as in start-igb-messagepng")
            Show
            cdias1 Chester Dias (Inactive) added a comment - Requested Changes have been made: https://bitbucket.org/chesterdias/genome-dashboard-local-chester/branch/IGBF-2731#diff
            Show
            cdias1 Chester Dias (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/genome-dashboard/pull-requests/27/igbf-2731
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged but not yet deployed to test.bioviz.org

            Show
            ann.loraine Ann Loraine added a comment - Merged but not yet deployed to test.bioviz.org
            Hide
            nfreese Nowlan Freese added a comment -

            [~aloraine] - have these changes been deployed to test.bioviz.org?

            Show
            nfreese Nowlan Freese added a comment - [~aloraine] - have these changes been deployed to test.bioviz.org?
            Hide
            ann.loraine Ann Loraine added a comment -

            Changes are deployed to 52.4.208.199.
            Updated DNS (just now) to associate address https://bioviztest.bioviz.org with 52.4.208.199.

            Show
            ann.loraine Ann Loraine added a comment - Changes are deployed to 52.4.208.199. Updated DNS (just now) to associate address https://bioviztest.bioviz.org with 52.4.208.199.
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Note: The host was updated to the latest branch and also includes code for IGBF-2731 "Detect Safari on Genome Dashboard"

            Show
            ann.loraine Ann Loraine added a comment - - edited Note: The host was updated to the latest branch and also includes code for IGBF-2731 "Detect Safari on Genome Dashboard"
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited

            Tested on macOS Safari Version 14.0.2. The warning banner appears when using safari and not when using chrome/firefox, as expected.

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited Tested on macOS Safari Version 14.0.2. The warning banner appears when using safari and not when using chrome/firefox, as expected.

              People

              • Assignee:
                cdias1 Chester Dias (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: