Re: Issue 1570 in reviewboard: Diff-Upload UI: Usability Enhancement

2014-07-24 Thread reviewboard


Comment #4 on issue 1570 by fbeach...@gmail.com: Diff-Upload UI:  Usability  
Enhancement

http://code.google.com/p/reviewboard/issues/detail?id=1570

Thanks to the RB team for thorough and patient attention to this report.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 1570 in reviewboard: Diff-Upload UI: Usability Enhancement

2010-03-24 Thread reviewboard

Status: New
Owner: 
Labels: Type-Enhancement Priority-Medium

New issue 1570 by fbeachler: Diff-Upload UI:  Usability Enhancement
http://code.google.com/p/reviewboard/issues/detail?id=1570

*NOTE: If you have a patch, please submit it to http://reviews.review-
board.org/


What version are you running?
1.0.1rc3


What's the URL of the page this enhancement relates to, if any?
New Review Request (step 1)/Upload Diff UIs

Describe the enhancement and the motivation for it.
I'm not sure if this enhancement is already planned for 1.5. I apologize if
this is a duplicate.

Stories:
1. As a user I want the order of the fields arranged in the DIFF upload
interface so the priority is presented as A) upload file, B) choose repo,
C) enter base path; so the priority of the fields reflects the priority of
the majority of use-cases (see motivation).
2. As a user I want the DIFF upload interface to be show me a list
(summary) of the file-paths in the patch as soon as enter a file for upload
so I can see if I'm choosing the right patch file and help me to calculate
the base path.
3. As a user I want the summary list in #2 to update dynamically as I make
choices from the Repository dropdown and enter text into the base path
field so I can see what the final calculated base path will be.

Motivation:
For a ReviewBoard user (who creates a new review request, or modifies the
DIFF of an existing one) - my first concern is choosing the patch file I
just created manually (e.g. 'what did I name it', 'where did I save
it'...).  My next concern is choosing the repository from the dropdown, and
then entering the Base Path for my DIFF.


What operating system are you using? What browser?
Mac/Windows, Firefox 3.5+/Safari 3+

Please provide any additional information below.

I'm not sure how to do provide the file summary feature with AJAX (I'm not
considering Flash/Silverlight here).  I think the file could be posted to
the server and expect a response that supports the functionality stated
above (i.e. a summary of the files in the patch).

This would have a positive impact on user experience regardless of what
technologies were used to build the UI.  For example, the user will wait
for the upload/summary process to complete, see if the file-path list looks
correct, choose the repo  enter the base path, and submit the form.  If
the base path is incorrect or files are not present in the repo (or another
error occurs trying to validate/generate the side-by-side) - the server
response can be much quicker and the summary of what may have caused the
error is in front of them.  As another example of improvement - if the
server dies before the user can submit their DIFF they'll know before even
trying to calculate or enter any other fields.  I believe more error
handling improvements could be realized but perhaps the repository value
must be known first...

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1570 in reviewboard: Diff-Upload UI: Usability Enhancement

2010-03-24 Thread reviewboard

Updates:
Status: Confirmed
Labels: Component-Reviews Usability

Comment #1 on issue 1570 by chipx86: Diff-Upload UI:  Usability Enhancement
http://code.google.com/p/reviewboard/issues/detail?id=1570

These are very good ideas. I think we should certainly provide this  
feedback.


It's unlikely that it can happen for 1.5. Probably not for 1.6 either, but  
if we do a

1.7 then maybe.

You may want to use post-review, which will help avoid the whole issue.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 1570 in reviewboard: Diff-Upload UI: Usability Enhancement

2010-03-24 Thread reviewboard


Comment #2 on issue 1570 by fbeachler: Diff-Upload UI:  Usability  
Enhancement

http://code.google.com/p/reviewboard/issues/detail?id=1570

When we started using RB that script worked for us.  For some reason when  
we started
using ssh+kerberos the script stopped working.  I'll check it again and  
either
refresh my team on the script or see if there's a reproducible and  
unreported issue.
 Anyway, now all users are very familiar with the manual process.  Finally,  
there's
the use-case of splitting a patch into multiple reviews.  Does post-review  
support that?


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.