Comments on: Patch-Tag Extends Free Trial, and other developments http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/ give your code a home Fri, 13 Jul 2012 17:30:25 +0000 hourly 1 http://wordpress.com/ By: Sergey http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-42 Fri, 12 Jun 2009 14:34:10 +0000 http://blog.patch-tag.com/?p=143#comment-42 Thank you! It’s better than nothing but still not perfect. Some remarks:

1) List of commits:

I often get 502 Proxy Error and the page loads slowly (http://patch-tag.com/r/happstack/snapshots/all/history)

Hashes are really long. Initial part of the hash duplicates the date. Probably it makes sense to visualize only the ending, like ‘…8fe174e57’ in the web UI.

Layout of the list of commits is less readable than on github or on bitbucket. There, hashes stay separately on the right and the list of updated files is shown only on the page with commit details. (http://github.com/ghc-hq/ghc/commits/master). But I think this will be polished in the future.

2) Commit page:

Striking out deleted text is not really a good idea: it is not clearly visible if the line strikes over the space or over the dash/minus sign. Better change background colour.

Browsing for changes through the whole big file is not very convenient. Diffs/patches are easier to read.

wdiffs are even better (github and bitbucket do it, really nice).

Line numbers may help in certain situations.

]]> By: Thomas Hartman http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-37 Sat, 23 May 2009 18:22:12 +0000 http://blog.patch-tag.com/?p=143#comment-37 OK, pushed!

What do you think of the new diff functionality?

You can click one of the “hash” type links at, eg

http://patch-tag.com/r/happstack/snapshots/all/history

to see it in action.

]]> By: Thomas Hartman http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-36 Tue, 19 May 2009 21:52:46 +0000 http://blog.patch-tag.com/?p=143#comment-36 Yeah, you’re right.

Just did some work on this. Testing, and will push something soon.

]]> By: Matthew Elder http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-35 Tue, 19 May 2009 16:26:23 +0000 http://blog.patch-tag.com/?p=143#comment-35 We do have file diffs (if you click on file history in the upper right-hand corner). But yes, we do need patch diffs, we will get this addressed asap.

]]> By: mornfall http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-34 Tue, 19 May 2009 13:52:27 +0000 http://blog.patch-tag.com/?p=143#comment-34 Dunno, so far the move away from gitit looks like a regression to me, in term of features (eg. the diffs already mentioned). Hopefully, that gets addressed in the future…

]]> By: Sergey http://blog.patch-tag.com/2009/05/12/patch-tag-extends-free-trial-and-other-developments/#comment-33 Fri, 15 May 2009 09:00:46 +0000 http://blog.patch-tag.com/?p=143#comment-33 Hi, Thomas!

I like Patch-Tag. Just two wishes:

1. Add a link to feature requests/bug tracking of the project somewhere (I mean patch-tag). For example, you can add a UserVoice (http://uservoice.com/) widget quickly.

2. It would be nice to be able to see highlighted patches (diffs) when browsing history. Now it’s impossible to see what patch does what.

]]>