My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 750: diff all side-by-side should work for images as well
3 people starred this issue and may be notified of changes. Back to list
Status:  Accepted
Owner:  ----


Sign in to add a comment
 
Reported by t...@google.com, Oct 8, 2010
When I'm reviewing asset drop cls, "diff all side-by-side" just shows a bunch of empty tabs.  But if i click on the "new" tab I actually get to see the image.  Don't know if the problem is that these new assets don't have a side-by-side diff or if it's because they're images.  Also, there aren't any tabs for the changes in the xml files.  Perhaps there's an upper limit on the number of tabs?

In any case, "diff all side-by-side" should be comprehensive:  I should be able to complete the review just by looking at the tabs, and there shouldn't be any affected files that don't show up in a tab.
May 20, 2011
Project Member #1 nas...@grainawi.org
(No comment was entered for this change.)
Status: Accepted
Jun 30, 2013
#2 krinklemail@gmail.com
See also bug 258.
Sign in to add a comment

Powered by Google Project Hosting