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

navigation links in repository view do not respect branch or tag context #797

Closed
gitblit opened this issue Aug 12, 2015 · 3 comments
Closed

Comments

@gitblit
Copy link
Collaborator

gitblit commented Aug 12, 2015

Originally reported on Google Code with ID 501

What steps will reproduce the problem?
1. Open the commit log of a branch (not the default one) of a repository (i.e. https://dev.gitblit.com/log/gitblit.git/refs!heads!develop)
2. In the navigation bar switch to "tree" tab

What is the expected output? What do you see instead?
What I expect: tree view of the current branch I am looking at (i.e. tree of branch
"develop": https://dev.gitblit.com/tree/gitblit.git/refs!heads!develop)
What I get: tree view of default branch (i.e. https://dev.gitblit.com/tree/gitblit.git)

This relates to most of the links in the navigation bar of a repository. In my opinion
"commits", "tree", "documentation" links should refer to the current branch.


What version of the product are you using? On what operating system?
1.6.0; debian 7.0

Please provide any additional information below.
This is a controversial issue as there seems to be a concept behind the navigation
rules. Some more information is probably needed for the context of the links in the
navigation bar, say i.e. some hint in a mouseover to which "branch / tag / ref" you
are switching by clicking a link button. Maybe an additional info box, showing the
current branch, tag or ref one is surfing, should be added to the repository views.


See also issue #473 (http://code.google.com/p/gitblit/issues/detail?id=473): there
is a patch suggesting the change proposed here (needs confirmation)




Reported by stephan.krull.ecg on 2014-09-11 07:53:11

@gitblit
Copy link
Collaborator Author

gitblit commented Aug 12, 2015

The patch for the ref preservation makes some unnecessary changes and - like the wildcard
patch - is clustered with completely unrelated changes.

I agree that the tree, commits, and docs links could/should preserve the ref.

Reported by James.Moger on 2014-09-11 14:18:59

@gitblit
Copy link
Collaborator Author

gitblit commented Aug 12, 2015

Fix merged to master.

The current commit id will be respected & preserved in the "commits", "tree", and "docs"
links *if* the id resolves to a branch or tag.  If the commit id does not resolve to
a branch or tag, then the id is ignored and those links will resolve to the default
branch; same as before.

Reported by James.Moger on 2014-09-12 13:27:40

  • Status changed: Queued
  • Labels added: Milestone-1.6.1

@gitblit
Copy link
Collaborator Author

gitblit commented Aug 12, 2015

v1.6.1 released

Reported by James.Moger on 2014-10-20 21:36:03

  • Status changed: Done

@gitblit gitblit closed this as completed Aug 12, 2015
@flaix flaix modified the milestone: 1.6.1 Dec 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants