Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Spring 2 2022 Jan 18 - Jan 28, Spring 3 2022 Jan 31 - Feb 11, Spring 4 2022 Feb 14 - Feb 25

      Description

      The top of the track hubs table page (see: https://www.bioviz.org/public-trackhubs.html) includes a lot of text that is not strictly necessary and uses up a lot of space.

      Modify the page top as shown in the attached image. (Do not make changes to the table itself as this would be done in a separate ticket.)

      Details:

      1. Headline uses imperative voice to emphasize that this interface is a tool that lets users view track hub data in IGB
      2. The headline and search box are both centered
      3. A note in smaller text left-justified in the column immediately to the right of the search box reads: "Want more info? Play video" with "Play video" hyperlinked to a youtube video. The youtube video is not yet created. For now, just link to the IGB YouTube channel.
      4. Note that the interface lacks clutter. A lot of interfaces contain extra words that increase the user's cognitive load with minimal benefit to them. Less is always more.

        Attachments

        1. 3063.png
          3063.png
          56 kB
        2. IGBF-3063.png
          IGBF-3063.png
          385 kB
        3. New top .png
          New top .png
          97 kB
        4. reduce-verticality.png
          reduce-verticality.png
          423 kB

          Activity

          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          I have made changes to the required HTML file and I am getting the expected output but the genome versions are not loading.

          Philip Badzuh Please let me know if that's expected.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited I have made changes to the required HTML file and I am getting the expected output but the genome versions are not loading. Philip Badzuh Please let me know if that's expected.
          Hide
          pbadzuh Philip Badzuh (Inactive) added a comment -

          Assuming you are serving the bioviz project locally, for the genome versions to load, you will need to also run the hub-facade server locally.

          Show
          pbadzuh Philip Badzuh (Inactive) added a comment - Assuming you are serving the bioviz project locally, for the genome versions to load, you will need to also run the hub-facade server locally.
          Hide
          omarne Omkar Marne (Inactive) added a comment -

          Thank you Philip Badzuh

          Show
          omarne Omkar Marne (Inactive) added a comment - Thank you Philip Badzuh
          Hide
          ann.loraine Ann Loraine added a comment -

          Maybe try "sup" tag?

          Show
          ann.loraine Ann Loraine added a comment - Maybe try "sup" tag?
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          I have modified the top of the table including the search bar and Question mark symbol.

          Please see my latest changes here

          Hello Dr. [~aloraine] Please check the snapshot and review the ticket.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited I have modified the top of the table including the search bar and Question mark symbol. Please see my latest changes here Hello Dr. [~aloraine] Please check the snapshot and review the ticket.
          Hide
          ann.loraine Ann Loraine added a comment -

          Can you modify the vertical placement of "Want more info? Play video" text to be centered? In the image file "screenshot-1.png", this line of text is too high in the line. Please see first image.

          Show
          ann.loraine Ann Loraine added a comment - Can you modify the vertical placement of "Want more info? Play video" text to be centered? In the image file "screenshot-1.png", this line of text is too high in the line. Please see first image.
          Hide
          omarne Omkar Marne (Inactive) added a comment -

          Dr. [~aloraine] Please check the snapshot if it looks good.

          Show
          omarne Omkar Marne (Inactive) added a comment - Dr. [~aloraine] Please check the snapshot if it looks good.
          Hide
          ann.loraine Ann Loraine added a comment -

          Thanks! It looks great to me – nicely symmetrical.

          Show
          ann.loraine Ann Loraine added a comment - Thanks! It looks great to me – nicely symmetrical.
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          Thank you, Dr. [~aloraine]! Please find my pull request here

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited Thank you, Dr. [~aloraine] ! Please find my pull request here
          Hide
          ann.loraine Ann Loraine added a comment -

          I checked the PR. It looks like you accidentally have added a number of files from your IDE-created local development environment. Please re-do the branch / commits so that these files are not introduced into the repository.
          attn: Omkar Marne

          Show
          ann.loraine Ann Loraine added a comment - I checked the PR. It looks like you accidentally have added a number of files from your IDE-created local development environment. Please re-do the branch / commits so that these files are not introduced into the repository. attn: Omkar Marne
          Hide
          omarne Omkar Marne (Inactive) added a comment -

          Hello Dr. [~aloraine] Please find my new PR here

          Show
          omarne Omkar Marne (Inactive) added a comment - Hello Dr. [~aloraine] Please find my new PR here
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Merged and deployed to bioviz main. Ready for testing.

          Show
          ann.loraine Ann Loraine added a comment - - edited Merged and deployed to bioviz main. Ready for testing.
          Hide
          ann.loraine Ann Loraine added a comment -

          It is looking good except for one detail.
          Please: Reduce vertical space as indicated in attached image.
          attn: Omkar Marne

          Show
          ann.loraine Ann Loraine added a comment - It is looking good except for one detail. Please: Reduce vertical space as indicated in attached image. attn: Omkar Marne
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          Hello Dr. [~aloraine], I have reduced the vertical space. The vertical space above the search bar looks more because 'the headline' is a different class and has its own vertical space.

          Please check if it looks good. I'll submit a new PR.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited Hello Dr. [~aloraine] , I have reduced the vertical space. The vertical space above the search bar looks more because 'the headline' is a different class and has its own vertical space. Please check if it looks good. I'll submit a new PR.
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Thank you for working on this Omkar Marne!

          Unfortunately, the distance between the top of the table and the first row of the header is very small and looks cramped compared to the vertical space around the other parts of the table. The vertical space needs to look more balanced.

          The reason this matters is that if there is something unbalanced about the appearance, users get distracted. Symmetry helps avoid this. It also looks more polished and professional, which gives users a feeling of comfort, important when displaying information.

          Philip Badzuh: Can you take a look?

          Maybe look at the "Igb genomes" page? Can you use some ideas from there?

          Show
          ann.loraine Ann Loraine added a comment - - edited Thank you for working on this Omkar Marne ! Unfortunately, the distance between the top of the table and the first row of the header is very small and looks cramped compared to the vertical space around the other parts of the table. The vertical space needs to look more balanced. The reason this matters is that if there is something unbalanced about the appearance, users get distracted. Symmetry helps avoid this. It also looks more polished and professional, which gives users a feeling of comfort, important when displaying information. Philip Badzuh : Can you take a look? Maybe look at the "Igb genomes" page? Can you use some ideas from there?
          Hide
          omarne Omkar Marne (Inactive) added a comment -

          Hello, Dr. [~aloraine] Please check now. I have reduced more space.

          Show
          omarne Omkar Marne (Inactive) added a comment - Hello, Dr. [~aloraine] Please check now. I have reduced more space.
          Hide
          ann.loraine Ann Loraine added a comment -

          It still looks unbalanced. Please compare to the mockup.

          Show
          ann.loraine Ann Loraine added a comment - It still looks unbalanced. Please compare to the mockup.
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          Hello Dr. [~aloraine] Please compare the changes with the mockup and let me know if it's good. Also, please let me know where should I add the 'Want more info' line. I have removed that line and text in the search bar.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited Hello Dr. [~aloraine] Please compare the changes with the mockup and let me know if it's good. Also, please let me know where should I add the 'Want more info' line. I have removed that line and text in the search bar.
          Hide
          ann.loraine Ann Loraine added a comment -

          Thank you so much for your patience. It looks great. Please submit a PR.

          Show
          ann.loraine Ann Loraine added a comment - Thank you so much for your patience. It looks great. Please submit a PR.
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          Dr. [~aloraine] Please find my pull request here.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited Dr. [~aloraine] Please find my pull request here .
          Hide
          ann.loraine Ann Loraine added a comment -

          PR is merged. New code is deployed. Please test.

          Show
          ann.loraine Ann Loraine added a comment - PR is merged. New code is deployed. Please test.
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          Testing instructions:

          • Open https://bioviz.org/index.html
          • Open Trackhub link.
          • The UI above the table should be the same as the image attached above. Please see the snapshot named 'New Top'.
          Show
          omarne Omkar Marne (Inactive) added a comment - - edited Testing instructions: Open https://bioviz.org/index.html Open Trackhub link. The UI above the table should be the same as the image attached above. Please see the snapshot named 'New Top'.
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Question:

          I'm a little confused about the testing instructions. The new page is deployed on bioviz.org, so I don't understand why the tester needs to run the back end locally in order to see the changes. The bioviz page is contacting translate.bioviz.org as the back end, so I don't understand why the tester needs to have the back end code running on their local, as well.

          Show
          ann.loraine Ann Loraine added a comment - - edited Question: I'm a little confused about the testing instructions. The new page is deployed on bioviz.org, so I don't understand why the tester needs to run the back end locally in order to see the changes. The bioviz page is contacting translate.bioviz.org as the back end, so I don't understand why the tester needs to have the back end code running on their local, as well.
          Hide
          omarne Omkar Marne (Inactive) added a comment - - edited

          You're right Dr. [~aloraine]. I have updated the testing instructions. You can directly go to bioviz.org and test it.

          Show
          omarne Omkar Marne (Inactive) added a comment - - edited You're right Dr. [~aloraine] . I have updated the testing instructions. You can directly go to bioviz.org and test it.
          Hide
          ann.loraine Ann Loraine added a comment -

          The new table looks great! Moving to Done.

          Show
          ann.loraine Ann Loraine added a comment - The new table looks great! Moving to Done.

            People

            • Assignee:
              omarne Omkar Marne (Inactive)
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: