Comment 5 for bug 1681943

Revision history for this message
Dan Scott (denials) wrote :

I don't think this is ready to be pushed; the commit introduces a lot of new CSS that should also be reflected in the RTL styles (float:left, margin-left, etc).

On a lesser note, I'm worried about the approach of making things look better on mobile by taking away content. This can be frustrating to those who now have no option for seeing the content that is hidden. That said, I have not yet offered up an alternative design or implementation, so I won't object to the commit being pushed on this particular basis.