FeedbackWorkshop » History » Version 7
Chris Cannam, 2011-03-16 06:32 PM
1 | 1 | Luis Figueira | h1. Feedback from the 16/March Workshop |
---|---|---|---|
2 | 1 | Luis Figueira | |
3 | 1 | Luis Figueira | h2. code.soundsoftware |
4 | 1 | Luis Figueira | |
5 | 1 | Luis Figueira | * No main link from front page http://www.soundsoftware.ac.uk/ |
6 | 6 | Chris Cannam | |
7 | 6 | Chris Cannam | _True, we should probably think about promoting it a bit more by now -- perhaps after this round of fixes!_ |
8 | 6 | Chris Cannam | |
9 | 1 | Luis Figueira | * Go to projects, search for "Easy" - not useful result (should give the project first) |
10 | 6 | Chris Cannam | |
11 | 6 | Chris Cannam | _True, I'm not sure what the ordering is -- the projects are there, but not listed first_ |
12 | 6 | Chris Cannam | |
13 | 1 | Luis Figueira | * On http://code.soundsoftware.ac.uk/projects/easyhg/wiki/InstallingEasyHg, it says that Windows release is highlighed on http://code.soundsoftware.ac.uk/projects/easyhg/files, but actually several are highlighted. |
14 | 6 | Chris Cannam | |
15 | 6 | Chris Cannam | _The install page is a stopgap for a proper website, agreed it's rather clumsy at the moment_ |
16 | 6 | Chris Cannam | |
17 | 1 | Luis Figueira | * On http://code.soundsoftware.ac.uk/projects/easyhg, the "Download" link is too far down, make it a big "Download" button at the top! |
18 | 6 | Chris Cannam | |
19 | 6 | Chris Cannam | _There is a download button at the top. Evidently prominence rather than height on the page is the issue here_ |
20 | 6 | Chris Cannam | |
21 | 1 | Luis Figueira | * On https://code.soundsoftware.ac.uk/projects/qm-test-project-mdp/settings, please add help to what different roles do. |
22 | 6 | Chris Cannam | |
23 | 6 | Chris Cannam | _Yes, this seems to have caused much confusion_ |
24 | 6 | Chris Cannam | |
25 | 1 | Luis Figueira | * Have an option to get sent email when added to a project |
26 | 6 | Chris Cannam | |
27 | 6 | Chris Cannam | _Definitely -- slightly surprised this didn't happen already_ |
28 | 6 | Chris Cannam | |
29 | 1 | Luis Figueira | * In Repository " This URL has Read+Write access." Is meaningless to user |
30 | 6 | Chris Cannam | |
31 | 6 | Chris Cannam | _This is issue #70_ |
32 | 6 | Chris Cannam | |
33 | 1 | Luis Figueira | * In Repository " your preferred Mercurial client" could be EasyMercurial. |
34 | 6 | Chris Cannam | |
35 | 6 | Chris Cannam | _Yes, ultimately_ |
36 | 6 | Chris Cannam | |
37 | 2 | Luis Figueira | * In Repository "Mercurial" is not a link |
38 | 6 | Chris Cannam | |
39 | 6 | Chris Cannam | _True, it should be_ |
40 | 6 | Chris Cannam | |
41 | 1 | Luis Figueira | * When empty, could say "no files yet"? |
42 | 6 | Chris Cannam | |
43 | 6 | Chris Cannam | _Possibly_ |
44 | 6 | Chris Cannam | |
45 | 1 | Luis Figueira | * Provide 2 URLs, like Doodle does |
46 | 6 | Chris Cannam | |
47 | 6 | Chris Cannam | _I don't know Doodle but apparently the suggestion is for "your URL" and "general URL to give to other people"_ |
48 | 6 | Chris Cannam | |
49 | 1 | Luis Figueira | * URL box easyhg is too short for https://markp@code.soundsoftware.ac.uk/hg/qm-test-project-mdp |
50 | 6 | Chris Cannam | |
51 | 6 | Chris Cannam | _Don't understand this one_ |
52 | 6 | Chris Cannam | |
53 | 1 | Luis Figueira | * On repository, e.g. https://code.soundsoftware.ac.uk/projects/qm-test-project-mdp/repository, when click on file name, allow "open" or "preview" for text files etc, not just "download" (which enforces save). |
54 | 6 | Chris Cannam | |
55 | 6 | Chris Cannam | _Don't understand this, the page has View and Annotate as well as Download_ |
56 | 6 | Chris Cannam | |
57 | 2 | Luis Figueira | * disable other languages |
58 | 6 | Chris Cannam | |
59 | 6 | Chris Cannam | _Right, or provide language fallbacks (I think this may be working in trunk code)_ |
60 | 1 | Luis Figueira | |
61 | 7 | Chris Cannam | * Entering a name [in Add User page for project] then typing enter gives an error message - reason code missing |
62 | 7 | Chris Cannam | |
63 | 7 | Chris Cannam | _Right, this is pretty weird -- form should just do nothing if no people are selected_ |
64 | 7 | Chris Cannam | |
65 | 1 | Luis Figueira | h2. EasyHg |
66 | 1 | Luis Figueira | |
67 | 1 | Luis Figueira | * On re-run (after install), it opens "Open Repository" dialog, but the default local Mercurial repostitory location is C:/Program Files (x86)/EasyMercurial, but should be e.g. C:/Users/[username]/My Documents (at least for 64-bit Windows 7). |
68 | 7 | Chris Cannam | |
69 | 7 | Chris Cannam | _This should be fixed already in Hg, but I haven't tested it on Windows yet_ |
70 | 7 | Chris Cannam | |
71 | 1 | Luis Figueira | * Remote repository defaults to " C:/Program Files (x86)/EasyMercurial" |
72 | 7 | Chris Cannam | |
73 | 7 | Chris Cannam | _Yes, this is quite wrong_ |
74 | 7 | Chris Cannam | |
75 | 1 | Luis Figueira | * Add text on the remote repository dialogue to instruct users to add the username to the remote repo URL "username@repository" |
76 | 7 | Chris Cannam | |
77 | 7 | Chris Cannam | _Better added to code.soundsoftware page I think?_ |
78 | 7 | Chris Cannam | |
79 | 1 | Luis Figueira | * Local folder defaults to C:/Users/markp, should default to C:/Users/markp/My Documents |
80 | 7 | Chris Cannam | |
81 | 7 | Chris Cannam | _This should be fixed in Hg_ |
82 | 7 | Chris Cannam | |
83 | 1 | Luis Figueira | * Local folder browse doesn't start from right place. E.g. Folder is " C:/Users/markp/My Documents/ ", click "Browse", shows C:\Program Files (x86)\EasyMercurial instead. |
84 | 7 | Chris Cannam | |
85 | 7 | Chris Cannam | _I'll review this_ |
86 | 7 | Chris Cannam | |
87 | 1 | Luis Figueira | * On windows, can't select a folder with "Browse" unless it exists, so why complain that it already exists? It now says I cannot clone to it! |
88 | 7 | Chris Cannam | |
89 | 7 | Chris Cannam | _This is #92, fixed in Hg_ |
90 | 7 | Chris Cannam | |
91 | 1 | Luis Figueira | * On cancel, Open Repository dialog forgets past state. |
92 | 7 | Chris Cannam | |
93 | 7 | Chris Cannam | _This may be trickier than it looks to fix_ |
94 | 7 | Chris Cannam | |
95 | 1 | Luis Figueira | * What "password" do I type? Is this for my PC or for the repository, or creating a new one for the project? |
96 | 7 | Chris Cannam | |
97 | 7 | Chris Cannam | _Good point, it should at least say "repository password" or something_ |
98 | 7 | Chris Cannam | |
99 | 1 | Luis Figueira | * Clone succsessful - more details is now wide enough (or should wrap text) |
100 | 7 | Chris Cannam | |
101 | 7 | Chris Cannam | _Low priority!_ |
102 | 7 | Chris Cannam | |
103 | 1 | Luis Figueira | * .hg directory not hidden on windows - should tell people to ignore it! |
104 | 7 | Chris Cannam | |
105 | 7 | Chris Cannam | _True... but where?_ |
106 | 7 | Chris Cannam | |
107 | 1 | Luis Figueira | * Commit to "local" repository - why? In Commit files dialog, maybe say its going to the local repository. Also, what is the "default branch"? |
108 | 7 | Chris Cannam | |
109 | 7 | Chris Cannam | _I think the first point here is part of the general background that the user simply needs to know. Commit in a DVCS always goes to the local repository, that's just about the first thing you should learn, and I think the user would very quickly tire of always being reminded of it._ |
110 | 7 | Chris Cannam | |
111 | 1 | Luis Figueira | * Commit doesn't use file selection, uses all files. (Can commit individually with right-click). |
112 | 7 | Chris Cannam | |
113 | 7 | Chris Cannam | _Correct, this is a recent intentional change -- using the selection was a cause of far more confusion in users who then couldn't see how to commit all files_ |
114 | 7 | Chris Cannam | |
115 | 1 | Luis Figueira | * keyboard shortcuts don't work on right-click menus, e.g. right-click button then "d" should to diff. |
116 | 7 | Chris Cannam | |
117 | 7 | Chris Cannam | _Wow, I didn't know that ever worked in menus. Agreed though, shortcuts are very lacking throughout_ |
118 | 7 | Chris Cannam | |
119 | 1 | Luis Figueira | * Push should warn on modified, uncommitted files. (Currently allows push with modified files). |
120 | 7 | Chris Cannam | |
121 | 7 | Chris Cannam | _True_ |
122 | 7 | Chris Cannam | |
123 | 1 | Luis Figueira | * Pushed 1 changeset dialog details text is too small. |
124 | 7 | Chris Cannam | |
125 | 7 | Chris Cannam | _Low priority -- all the "More Details" texts are supposed to be for troubleshooting only really_ |
126 | 7 | Chris Cannam | |
127 | 1 | Luis Figueira | * Folder dialog uses "/" not "\"? |
128 | 7 | Chris Cannam | |
129 | 7 | Chris Cannam | _Not sure which dialog is referred to here? Windows actually uses / and \ interchangeably internally so this is never "wrong", but granted it's probably not helpful either_ |
130 | 7 | Chris Cannam | |
131 | 1 | Luis Figueira | * Why does "open" say "Clone successful"? Does "open" = "Clone"? |
132 | 7 | Chris Cannam | |
133 | 7 | Chris Cannam | _Very good point_ |
134 | 7 | Chris Cannam | |
135 | 7 | Chris Cannam | * Commit files dialog box doesn't close on enter. Also can't use TAB to get to enter button. What's keyboard shortcut to commit? |
136 | 7 | Chris Cannam | |
137 | 7 | Chris Cannam | _It certainly doesn't close on enter -- that would be very dangerous! Not being able to tab is a problem though, I agree. There's no shortcut for it in this dialog at the moment._ |
138 | 7 | Chris Cannam | |
139 | 1 | Luis Figueira | * On commit, I get Command Failed: |
140 | 1 | Luis Figueira | <pre> |
141 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
142 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
143 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
144 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
145 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
146 | 1 | Luis Figueira | warning: code.soundsoftware.ac.uk certificate not verified (check web.cacerts config setting) |
147 | 1 | Luis Figueira | abort: authorization failed |
148 | 1 | Luis Figueira | |
149 | 1 | Luis Figueira | If not authorized, suggest ask the project manager to make sure you have access rights. |
150 | 1 | Luis Figueira | |
151 | 7 | Chris Cannam | _That's quite a good idea_ |
152 | 7 | Chris Cannam | |
153 | 1 | Luis Figueira | </pre> |
154 | 1 | Luis Figueira | |
155 | 1 | Luis Figueira | * it would be important to store the password for the session duration; |
156 | 7 | Chris Cannam | |
157 | 7 | Chris Cannam | _This is what the "Keyring Extension":http://mercurial.selenic.com/wiki/KeyringExtension is for, but I haven't had time to integrate it properly (or instruct people in how to enable it themselves)_ |
158 | 7 | Chris Cannam | |
159 | 1 | Luis Figueira | * is it possible to have an easy way to change between projects. |
160 | 7 | Chris Cannam | |
161 | 7 | Chris Cannam | _We should probably add File -> Recent Repositories, though in fact going through the Open dialog is less onerous than you might think_ |
162 | 7 | Chris Cannam | |
163 | 1 | Luis Figueira | * when you have uncommitted changes and push, the dialogue shou warn you that you are about to push even though you have uncommitted changes. |
164 | 7 | Chris Cannam | |
165 | 7 | Chris Cannam | _True (this is also mentioned further up)_ |
166 | 7 | Chris Cannam | |
167 | 1 | Luis Figueira | * History Tab: the working revision black border is not explicit enough. |
168 | 7 | Chris Cannam | |
169 | 7 | Chris Cannam | _OK... what shall we do about this one?_ |
170 | 7 | Chris Cannam | |
171 | 1 | Luis Figueira | * When working in a previous revision, the Update button is enabled; maybe it should read "Update to tip of branch" (some users found it confusing Updating to a revision and still having the Update button enabled) |
172 | 7 | Chris Cannam | |
173 | 7 | Chris Cannam | _The status line tip for it does actually say something to this effect already. Have to check whether the tooltip can be made to match, or whether it always follows the (limited space) text_ |
174 | 2 | Luis Figueira | |
175 | 3 | Luis Figueira | h2. General Comments |
176 | 4 | Luis Figueira | |
177 | 2 | Luis Figueira | * Video on Merge command |
178 | 2 | Luis Figueira | * some questions we should answer in the documentation: |
179 | 2 | Luis Figueira | ** where should I put the local folder? |
180 | 2 | Luis Figueira | ** is a Mercurial folder different from a normal working folder? |
181 | 2 | Luis Figueira | ** advantages of local repo?? |
182 | 2 | Luis Figueira | ** how often should I commit? |
183 | 2 | Luis Figueira | ** how often should I push?? |
184 | 5 | Chris Cannam | * Some confusion noticed over Redmine roles -- e.g. what's the point of Reporter? Can we document these better in add-user interface? |
185 | 5 | Chris Cannam | |
186 | 5 | Chris Cannam | h2. People who had problems running the thing at all |
187 | 5 | Chris Cannam | |
188 | 5 | Chris Cannam | * Maria, Andrew N -- similar but not identical PySomeSymbol missing errors from extension |
189 | 5 | Chris Cannam | * Mike -- some problem from extension, error not noted |
190 | 5 | Chris Cannam | * Xue -- stops responding (on windows) when clicking OK on settings dialog |
191 | 5 | Chris Cannam | |
192 | 5 | Chris Cannam | h2. Other |
193 | 5 | Chris Cannam | |
194 | 5 | Chris Cannam | * Xue tried: Commit; commit; update back to first commit; edit again -> easyhg asked for a merge, but it wasn't shown as requiring one at all in history |