JSLint is complaining that (true) is a weird condition. Which is understandable if I wasn\'t using it on a reversed switch statement. So is JSLint
numberOfColumns = Math.max(4, Math.floor(menuLinksLength / 3));
This will give you identical results to your existing code. Your question is fairly ambiguous, as "acceptable" is a very subjective term. I would personally reject any merge request with a reversed switch statement, because I actually can't think of a situation where that couldn't be replaced with something simpler and/or easier to read.