Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance usability for ticgitweb #76

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Enhance usability for ticgitweb #76

wants to merge 3 commits into from

Conversation

jandechent
Copy link

The web interface should show which repository is currently displayed. This is achieved by showing the directory in the title and in the html body. This is especially important if several instanced of ticgitweb are active.

Fup Duck and others added 3 commits September 14, 2012 08:15
…s of ticgitweb can be running, but there is no easy way to distinguish between them.
@jeffWelling
Copy link
Owner

Hey!
Thanks for the pull request, you've got a good point about the ticgitweb program.
Because it doesn't fix a critical bug I don't think it would be appropriate to push this out as a hotfix but I will definitely include this in the next non-hotfix release, which should be done in the next couple months and should bring some interesting changes.

Also, please note that the ticgitweb program is quite poorly written and needs to be replaced. I'm glad that you're able to make use of it the way it is (please don't have it open in more than one tab, if you go to do a commit in one and something else in another, I cannot promise your house won't burn down around you), just be aware that it isn't safe for more than one person to use (or more than one tab) in it's current form.

Thanks again, I'll pull this into the deving branch shortly :)

@jandechent
Copy link
Author

Thanks for the advice regaring ticgitweb. Fortunately we need it mainly for
documentatin purposes and not for massive bug-tracking.

On 5 October 2012 20:55, Jeff Welling [email protected] wrote:

Hey!
Thanks for the pull request, you've got a good point about the ticgitweb
program.
Because it doesn't fix a critical bug I don't think it would be
appropriate to push this out as a hotfix but I will definitely include this
in the next non-hotfix release, which should be done in the next couple
months and should bring some interesting changes.

Also, please note that the ticgitweb program is quite poorly written and
needs to be replaced. I'm glad that you're able to make use of it the way
it is (please don't have it open in more than one tab, if you go to do a
commit in one and something else in another, I cannot promise your house
won't burn down around you), just be aware that it isn't safe for more than
one person to use (or more than one tab) in it's current form.

Thanks again, I'll pull this into the deving branch shortly :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/76#issuecomment-9186010.

Dipl. Phys. Jan Falk Dechent

(1) Max Planck Institut für Polymerforschung
Ackermannweg 10, 55128 Mainz, Deutschland
Tel: +49(6131)379-216, Fax: +49(6131)379-100
http://www.mpip-mainz.mpg.de/groups/spiess

(2) Bereich Medizinische Physik (605, -1.311)
Klinik für Diagnostische und Interventionelle Radiologie,
Universitätsmedizin der Johannes Gutenberg-Universität
Mainz,
Langbeckstraße 1, 55131 Mainz, Deutschland
Tel. +49(6131)17-4298
http://www.medizinische-physik.klinik.uni-mainz.de

@ghost ghost assigned jeffWelling Jun 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants