Uploaded image for project: 'Bitbucket Cloud'
  1. Bitbucket Cloud
  2. BCLOUD-15110

Use sticky/floating header to indicate filename when scrolling large diff in PR/commit

    XMLWordPrintable

Details

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

    Description

      When reviewing a pull request's diff, if one file is changed extensively, I often have to scroll through quite a few lines while reviewing the changes to that file. Sometimes I forget what file I'm looking at by the time I get to the end, and I have to scroll back up to the top of the file to see the header indicating the file path. This precision scroll jump can be difficult if there are lots of files in the diff; if you go too quickly, you can miss the separation padding between files. Then I have to jump back down to the next file to continue reviewing.

      I suggest floating the file header with the file path so that it is always visible, or "sticky" to the top of the screen. When you begin scrolling through a diff, eventually the diff of first file listed will begin to move offscreen. When this happens, the filename header should remain onscreen by sliding down from its original position. Only when the file diff is completely scrolled offscreen should the filename also scroll offscreen.

      Attachments

        Activity

          People

            Unassigned Unassigned
            2309b0756957 ansonj
            Votes:
            2 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: