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

      Many users have requested a way to toggle the "ignore whitespace" option directly from the UI. This is a follow-on from BCLOUD-6024 which provides a query param for ignoring whitespace, but isn't sticky (meaning that it needs to be re-applied to each page view and doesn't follow the user through navigation) and requires manually appending the query param to the url.

          Form Name

            [BCLOUD-11864] "Ignore whitespace" toggle in the UI (BB-14331)

            Hi Jeff,

            Please file a new ticket with details on any additional enhancements you'd like. The overwhelming majority of feedback on this implementation has been positive.

            Cheers,
            Zach

            Zachary Davis (Inactive) added a comment - Hi Jeff, Please file a new ticket with details on any additional enhancements you'd like. The overwhelming majority of feedback on this implementation has been positive. Cheers, Zach

            Jeff Lomax added a comment -

            Preferable over nothing, yes, but Issue Resolved, NO.

            Jeff Lomax added a comment - Preferable over nothing, yes, but Issue Resolved, NO.

            Shaun Ek added a comment -

            Preferable over nothing for sure, thanks guys.

            Shaun Ek added a comment - Preferable over nothing for sure, thanks guys.

            Hi everyone,

            We've just released a button on every file in a diff that toggles ignore whitespace for all files in the diff. This option is currently not sticky (the setting isn't remembered on reload or navigation), and there is no account or repo-level setting. Those are both things we considered and explored but ultimately we felt it was better to get this functionality out as it currently stands.

            If you run into any bugs, please let us know, . If you want enhancements, please file new tickets.

            Thanks for your patience,
            Zach

            Zachary Davis (Inactive) added a comment - Hi everyone, We've just released a button on every file in a diff that toggles ignore whitespace for all files in the diff. This option is currently not sticky (the setting isn't remembered on reload or navigation), and there is no account or repo-level setting. Those are both things we considered and explored but ultimately we felt it was better to get this functionality out as it currently stands. If you run into any bugs, please let us know, . If you want enhancements, please file new tickets. Thanks for your patience, Zach

            klehrke added a comment -

            You could always let the dev know and help out. He's always looking to make it better / more helpful. That also doesn't happen to me. Redirect happens once for the white space button, but the diff toggle doesn't seem to redirect.

            klehrke added a comment - You could always let the dev know and help out. He's always looking to make it better / more helpful. That also doesn't happen to me. Redirect happens once for the white space button, but the diff toggle doesn't seem to redirect.

            alexk-cenx added a comment -

            @klehrke Yes & no, I have that plugin installed and it is better than nothing, but it causes so many redirects before the page settles down that a PR is often unusable for 5 seconds after opening it. During that time I can't interact with anything on the page (edit, approve, comments...).

            alexk-cenx added a comment - @klehrke Yes & no, I have that plugin installed and it is better than nothing, but it causes so many redirects before the page settles down that a PR is often unusable for 5 seconds after opening it. During that time I can't interact with anything on the page (edit, approve, comments...).

            klehrke added a comment -

            You guys should check out the chrome plugin my friend made. It has two main features, first a toggle switch for collapsing file diffs for annoying long diffs. Also, a button for doing the w=1.

            https://chrome.google.com/webstore/detail/bitbucket-pull-request-to/hfebajohpclnfhfnlhgndbmcdnlchjjd

            klehrke added a comment - You guys should check out the chrome plugin my friend made. It has two main features, first a toggle switch for collapsing file diffs for annoying long diffs. Also, a button for doing the w=1. https://chrome.google.com/webstore/detail/bitbucket-pull-request-to/hfebajohpclnfhfnlhgndbmcdnlchjjd

            SeBsZ added a comment -

            I didn't even know this GET parameter existed! I've been wondering why Bitbucked wasn't ignoring whitespace in diffs. Please make this a sticky toggle!

            SeBsZ added a comment - I didn't even know this GET parameter existed! I've been wondering why Bitbucked wasn't ignoring whitespace in diffs. Please make this a sticky toggle!

            archon810 added a comment -

            I've been ever since I started using Bitbucket wondering why in the world it doesn't have an ignore whitespace flag for diffs, and today I found out that it actually does exist https://blog.bitbucket.org/2015/01/30/new-year-new-features but is completely hidden away in the UI. Users are expected to append w=1 to the url.

            This is such a useful feature, and hiding it just doesn't make sense. Please bring it to the UI (check out the comments on the blog post too).

            archon810 added a comment - I've been ever since I started using Bitbucket wondering why in the world it doesn't have an ignore whitespace flag for diffs, and today I found out that it actually does exist https://blog.bitbucket.org/2015/01/30/new-year-new-features but is completely hidden away in the UI. Users are expected to append w=1 to the url. This is such a useful feature, and hiding it just doesn't make sense. Please bring it to the UI (check out the comments on the blog post too).

            David, yes agreed. Maybe in 2016 we can hope to be diffing like it's 1996.

            nigel_stewart_roames added a comment - David, yes agreed. Maybe in 2016 we can hope to be diffing like it's 1996.

              Unassigned Unassigned
              zdavis Zachary Davis (Inactive)
              Votes:
              42 Vote for this issue
              Watchers:
              44 Start watching this issue

                Created:
                Updated:
                Resolved: