berkus / rietveld

Code review tool from Guido for Django
code.google.com/p/rietveld
Apache License 2.0
0 stars 0 forks source link

Patch set summary table is 2000 pixels wide #500

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Open an issue in Rietveld

The patch set summary table is wider than my widescreen monitor, making the 
code review process nearly impossible. This problem started yesterday 
(8/7/2014); before that there was no problem. I tried restarting my browser, 
which is Chrome on Kubuntu 14.04.

URL e.g.: https://codereview.appspot.com/117340043/

Screenshot is attached. Please let me know if there is a workaround.

Original issue reported on code.google.com by byron.c....@gmail.com on 8 Aug 2014 at 6:57

Attachments:

GoogleCodeExporter commented 9 years ago
Yea, this is annoying. I think .issue-description should be "white-space: 
pre-wrap" instead of just "white-space: pre"

Original comment by est...@chromium.org on 21 Aug 2014 at 12:49

GoogleCodeExporter commented 9 years ago
caused by https://codereview.appspot.com/114340043/

Original comment by est...@chromium.org on 21 Aug 2014 at 12:53

GoogleCodeExporter commented 9 years ago
here's an untested patch

Original comment by est...@chromium.org on 21 Aug 2014 at 1:00

Attachments:

GoogleCodeExporter commented 9 years ago
Yeah, I was noticing this too.  I'll try to land and deploy this tomorrow.

Original comment by jrobbins@google.com on 21 Aug 2014 at 2:40

GoogleCodeExporter commented 9 years ago
I can't see anything.

Original comment by techtonik@gmail.com on 21 Aug 2014 at 2:40

GoogleCodeExporter commented 9 years ago
Committed as rev f9b3c06649c3ba73799cb418078ec7d800cc8311

This will be live in the next release, which I am planning on doing later today.

Original comment by jrobbins@google.com on 21 Aug 2014 at 4:27

GoogleCodeExporter commented 9 years ago
Correction: to avoid conflict with to other planned deployments, I'll hold off 
on deploying this until next week.

Original comment by jrobbins@google.com on 21 Aug 2014 at 10:43

GoogleCodeExporter commented 9 years ago
This change is live now.

Original comment by jrobbins@google.com on 25 Aug 2014 at 5:50