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

Change Show Console to Show Log

        Details

        • Story Points:
          1
        • Epic Link:
        • Sprint:
          Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19, Winter 5 Feb 22 - Mar 5

          Description

          Situation: In IGBF-2550 we renamed the Console tab to now be called Log. To be consistent, we should also change the Show Console dropdown under Help.

          Task: Change Show Console to Show Log under the Help dropdown.

            Attachments

              Issue Links

              relates to

              Improvement - An improvement or enhancement to an existing feature or task. IGBF-2550 Rename IGB Console tab

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                inaylor Irvin Naylor (Inactive) added a comment -

                Nowlan Freese Question: With the renaming, should we also rename everything in the backend that points to "ShowConsole" to "ShowLog" as well, or is this only a front-end renaming?

                Show
                inaylor Irvin Naylor (Inactive) added a comment - Nowlan Freese Question: With the renaming, should we also rename everything in the backend that points to "ShowConsole" to "ShowLog" as well, or is this only a front-end renaming?
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Great question, so I'd like to weigh in:
                I do think we ought to change the back end variable names to improve readability of the code, but we should do that in a separate ticket. We can also label the ticket with "Beginner" as this could be a good task for one of the newer joiners.

                Show
                ann.loraine Ann Loraine added a comment - Great question, so I'd like to weigh in: I do think we ought to change the back end variable names to improve readability of the code, but we should do that in a separate ticket. We can also label the ticket with "Beginner" as this could be a good task for one of the newer joiners.
                Hide
                Permalink
                inaylor Irvin Naylor (Inactive) added a comment -

                Sounds good!
                Here's the link to my branch: https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-2764/

                Show
                inaylor Irvin Naylor (Inactive) added a comment - Sounds good! Here's the link to my branch: https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-2764/
                Hide
                Permalink
                rweidenh Logan Weidenhammer (Inactive) added a comment -

                Functional Review:
                Download and Install works.
                Help tab has the option "Show Log" and not 'Show Console".
                It functions appropriately.

                Code Review:
                Changes look good: substituted the word "Log" for "Console".

                Recommending Pull Request.

                Show
                rweidenh Logan Weidenhammer (Inactive) added a comment - Functional Review: Download and Install works. Help tab has the option "Show Log" and not 'Show Console". It functions appropriately. Code Review: Changes look good: substituted the word "Log" for "Console". Recommending Pull Request.
                Hide
                Permalink
                inaylor Irvin Naylor (Inactive) added a comment -

                Pull request is up: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/861/igbf-2764-change-show-console-to-show-log

                Show
                inaylor Irvin Naylor (Inactive) added a comment - Pull request is up: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/861/igbf-2764-change-show-console-to-show-log
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Merged. Installers are ready for testing.

                Show
                ann.loraine Ann Loraine added a comment - Merged. Installers are ready for testing.
                Hide
                Permalink
                nfreese Nowlan Freese added a comment -

                Tested on Mac 10.14.6 with dmg installer.

                Show Console now appears as Show Log.

                Closing issue.

                Show
                nfreese Nowlan Freese added a comment - Tested on Mac 10.14.6 with dmg installer. Show Console now appears as Show Log. Closing issue.

                  People

                  • Assignee:
                    inaylor Irvin Naylor (Inactive)
                    Reporter:
                    nfreese Nowlan Freese
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: