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

Update BioViz Connect page on bioviz.org

    Details

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

      Description

      Situation: We need to remove the button/link to the reviewer video for BioViz Connect on bioviz.org/connect.html. We could replace the video with a link to the webinar video from CyVerse. Or we can redesign the page to make both videos available.

      Task: Come up with a design for the bioviz.org/connect.html page and once approved, implement the changes.

        Attachments

          Issue Links

            Activity

            Hide
            nfreese Nowlan Freese added a comment -

            Link to YouTube video: https://www.youtube.com/watch?v=YoZX4HlkHHA

            iframe/embed code from Tina:

            <iframe width="560" height="315" src="https://www.youtube.com/embed/YoZX4HlkHHA" title="YouTube video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>

            Show
            nfreese Nowlan Freese added a comment - Link to YouTube video: https://www.youtube.com/watch?v=YoZX4HlkHHA iframe/embed code from Tina: <iframe width="560" height="315" src="https://www.youtube.com/embed/YoZX4HlkHHA" title="YouTube video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
            Hide
            nfreese Nowlan Freese added a comment - - edited

            Things to include:

            1. Link to YouTube video This video is too old
            2. Link to CyVerse Webinar video
            3. Link to BioViz Connect paper
            Show
            nfreese Nowlan Freese added a comment - - edited Things to include: Link to YouTube video This video is too old Link to CyVerse Webinar video Link to BioViz Connect paper
            Hide
            karthik Karthik Raveendran added a comment -

            Changes pushed. See Commit
            New UI image is attached to this issue.

            Show
            karthik Karthik Raveendran added a comment - Changes pushed. See Commit New UI image is attached to this issue.
            Hide
            nfreese Nowlan Freese added a comment - - edited

            Changes requested (see nowlanV2.png):

            • More white space between "Create a free CyVerse account here"and the YouTube video.
            • More white space between YouTube video and biovizConnect.png image.
            • Make sure connect is italicized in reference to BioViz Connect paper and in description of YouTube video.
            • Shorten video description to "Watch our BioViz Connect webinar hosted by CyVerse on September 23, 2022."
            • Remove hr line.

            Overall looks really good on my laptop (see nowlanV2_laptop.png). I think it could use a little more white space separation, for example the youtube video description seemed more like a title to the image below it. Not sure about the hr line, I think with the additional whitespace there is a clear distinction between sections.

            Show
            nfreese Nowlan Freese added a comment - - edited Changes requested (see nowlanV2.png): More white space between "Create a free CyVerse account here"and the YouTube video. More white space between YouTube video and biovizConnect.png image. Make sure connect is italicized in reference to BioViz Connect paper and in description of YouTube video. Shorten video description to "Watch our BioViz Connect webinar hosted by CyVerse on September 23, 2022." Remove hr line. Overall looks really good on my laptop (see nowlanV2_laptop.png). I think it could use a little more white space separation, for example the youtube video description seemed more like a title to the image below it. Not sure about the hr line, I think with the additional whitespace there is a clear distinction between sections.
            Hide
            nfreese Nowlan Freese added a comment -
            Show
            nfreese Nowlan Freese added a comment - New commit: https://bitbucket.org/KarthikRavee91/bioviz/commits/e896950933085efd297747c99335f6d25677de08 Changes look good. Ready for pull request.
            Show
            nfreese Nowlan Freese added a comment - Pull request: https://bitbucket.org/lorainelab/bioviz/pull-requests/105
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged. Code deployed to BioViz.org "main" site. Ready for testing.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged. Code deployed to BioViz.org "main" site. Ready for testing.
            Hide
            nfreese Nowlan Freese added a comment -

            Testing on Mac using Firefox.

            All of the links work correctly, the embedded video works correctly, and I was able to log in to BioViz Connect.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Testing on Mac using Firefox. All of the links work correctly, the embedded video works correctly, and I was able to log in to BioViz Connect. Closing ticket.

              People

              • Assignee:
                karthik Karthik Raveendran
                Reporter:
                nfreese Nowlan Freese
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: