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

Ability to ignore whitespace changes in the diffs (BB-11120)

XMLWordPrintable

    • 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.

      I would love the ability to have an "Ignore whitespace" toggle on the popup side-by-side diff viewer.

      Nowadays, we utilize the side-by-side diff to more clearly see what changed in a changeset or pull request. We love it. Great stuff. But sometimes we get a contribution from a boneheaded developer who decided to use his IDE's "format" functionality to format the code according to his personal preference, and this can make it very difficult for us to spot the actual changes in the file, as opposed to just whitespace changes.

      Yes, the side-by-side diff viewer popup thingy does enable us to better see the whitespace changes than the standard diff output, but adding an Ignore Whitespace toggle would be absolutely wonderful.

        1. 1022493330-thumbsup.jpg
          47 kB
          Legacy Bitbucket Cloud User
        2. 4263911919-Pasted%20image%20at%202016_04_05%2012_34.png
          10 kB
          Shaun Ek

            [BCLOUD-6024] Ability to ignore whitespace changes in the diffs (BB-11120)

            Stephen Wrathall added a comment - - edited

            According to this page, it looks like this used to be a feature by adding `?w=1` to the request URL of a single commit:
            https://bitbucket.org/blog/new-year-new-features 
            see: "Ignore whitespace in diffs via URL" heading.

            But it doesn't work any more.

            Stephen Wrathall added a comment - - edited According to this page, it looks like this used to be a feature by adding ` ?w=1 ` to the request URL of a single commit: https://bitbucket.org/blog/new-year-new-features   see: "Ignore whitespace in diffs via URL" heading. But it doesn't work any more.

            Ian Bishop added a comment - - edited

            Not sure when it was added, but it is possible to ignore whitespace in PR diffs by clicking the 'Settings' button (next to approve) and unselecting 'Whitespace changes'

            Ian Bishop added a comment - - edited Not sure when it was added, but it is possible to ignore whitespace in PR diffs by clicking the 'Settings' button (next to approve) and unselecting 'Whitespace changes'

            Sebastián Vansteenkiste added a comment -

            is this "closed" as in "it's not going to be implemented"? It's really a pain to not be able to ignore whitespaces in PRs!!

            Sebastián Vansteenkiste added a comment - is this "closed" as in "it's not going to be implemented"? It's really a pain to not be able to ignore whitespaces in PRs!!

            Shaun Ek added a comment -

            Attachment 4263911919-Pasted%20image%20at%202016_04_05%2012_34.png has been added with description: Originally embedded in Bitbucket issue #6024 in site/master

            Shaun Ek added a comment - Attachment 4263911919-Pasted%20image%20at%202016_04_05%2012_34.png has been added with description: Originally embedded in Bitbucket issue #6024 in site/master

            Legacy Bitbucket Cloud User (Inactive) added a comment -

            Attachment 1022493330-thumbsup.jpg has been added with description: Originally embedded in Bitbucket issue #6024 in site/master

            Legacy Bitbucket Cloud User (Inactive) added a comment - Attachment 1022493330-thumbsup.jpg has been added with description: Originally embedded in Bitbucket issue #6024 in site/master

            roytzur added a comment -

            Olivier Bellemare, you're right - it does. Though the ui is confusing because you choose "Ignore whitespace" for a file and not for a PR/branch.......

            roytzur added a comment - Olivier Bellemare, you're right - it does. Though the ui is confusing because you choose "Ignore whitespace" for a file and not for a PR/branch.......

            xob added a comment -

            Roy Tzur, isn't that already what it does? Or have I been mistaken for a year and a half?

            xob added a comment - Roy Tzur, isn't that already what it does? Or have I been mistaken for a year and a half?

            roytzur added a comment -

            What about ignoring the white spaces of all the files in a specific PR? Do we have to do it one by one?

            roytzur added a comment - What about ignoring the white spaces of all the files in a specific PR? Do we have to do it one by one?

            Jason Mathison added a comment -

            If you would like to vote for "Ignore whitespace" to include ignoring deleted lines, vote here: https://bitbucket.org/site/master/issues/14316

            Jason Mathison added a comment - If you would like to vote for "Ignore whitespace" to include ignoring deleted lines, vote here: https://bitbucket.org/site/master/issues/14316

            nigel_stewart_roames added a comment -

            C'mon Atlassian, diffing is important. Take it seriously.

            nigel_stewart_roames added a comment - C'mon Atlassian, diffing is important. Take it seriously.

              Unassigned Unassigned
              32f738d5448a Shaun Ek
              Votes:
              180 Vote for this issue
              Watchers:
              111 Start watching this issue

                Created:
                Updated:
                Resolved: