Skip to content

Fix broken "Show whitespace changes" button on MR "Changes" tab

Jacques Erasmus requested to merge 52122-fix-broken-whitespace-button into master

This MR fixes the issue where the Show/Hide whitespace changes button was getting stuck in the Show whitespace changes state once it has been toggled.

The problem was that the w=0 parameter in the URL was getting handled as if it was an integer flag. Since url params are always strings the flag will always result in true regardless of the value.

This MR fixes the issue by converting the w flag to an integer when it is read out.

Tests for the component has also been added.

What are the relevant issue numbers?

Closes #52122 (closed)

Does this MR meet the acceptance criteria?

Edited by Jacques Erasmus

Merge request reports