Hi Terry.
Thanks for reporting these. Comments inline.
On Sun, Dec 28, 2008 at 3:28 PM, Terry Zhong think...@gmail.com wrote:
Hi Christian,
Nice work for new ui! However bugs spotted. Upgraded our testing
server 30min ago, new ui looks great except:
1. according to bug #656 Opera should be able to click on file name to
jump to diff. In previous version when I click, the browser trembles a
little and keep on top, and now the browser just keep still, as if I
didn't click. IE is OK.
I'll look into this. It was working before.
2. There are something wrong in star toggling scripts. When I go to /
r/, there is a script error on line 211, which is
$(#star-reviewrequests-75).toggleStar(
The new star script only works in dashboard, not here. Both IE and
Opera.
Thanks. This is now fixed in r1652.
3. When viewing any diff in IE, no matter review quest or diff view
(espcially diff view), CPU is always close to 100%. Opera is OK.
Strange, I hadn't noticed this. Will look into this.
4. The browser should break very long line into several short lines to
fit screen. However now in IE long lines are not broken, and the whole
window is a wide mess. I can hardly provide a screenshop since it's
very very wide. Opera is OK.
This is an older bug (
http://code.google.com/p/reviewboard/issues/detail?id=634). I don't think
it's actually new in this UI rewrite. Unless it's really a different issue.
5. Some strange black box appear in diff view in IE, most time
surrounding the title of diff for each file. Randomly, and usually
only occur at the first file diff block. Opera is OK.
That's the new selection border. It used to be that clicking a section of
the diff, or using navigation keys such as N and P, would select that
section by showing an arrow in the left-hand column. We now use a bold
border to show this, since not enough people noticed the arrow and
complained.
If this is something different, can you provide a screenshot?
6. There is a view button in comment dialog,however I spotted that
when I click it, the browser didn't position precisely at the comment,
because the new Loading box will change page height dynamically, and
the position is lost.
Huh, thought I had worked around this. I'll look into it.
I can reproduce these for several times so I guess they really exist.
If you need any more details, please let me know, glad to help.
Testing environment:
Django 9690
Djblets 11923
RB 1651
Opera 9.63
IE7
Server, browser and memcached are all cleared.
Terry Zhong
Thanks! We'll work to resolve these.
Christian
--
Christian Hammond - chip...@chipx86.com
VMware, Inc.
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---