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

Add missing History link to directory listings #4754

Closed
wants to merge 4 commits into from
Closed

Add missing History link to directory listings #4754

wants to merge 4 commits into from

Conversation

decke
Copy link
Contributor

@decke decke commented Aug 21, 2018

The feature to list commits from a subdirectory is already there but
so far the history link to it was missing. There is a History button
in the view_file.tmpl already so avoid showing two history buttons in
that case.

The GitHub webinterface has the same History button in the same place
so this makes gitea a little bit more compatible.

Signed-off-by: Bernhard Froehlich decke@bluelife.at

The feature to list commits from a subdirectory is already there but
so far the history link to it was missing. There is a History button
in the view_file.tmpl already so avoid showing two history buttons in
that case.

The GitHub webinterface has the same History button in the same place
so this makes gitea a little bit more compatible.

Signed-off-by: Bernhard Froehlich <decke@bluelife.at>
@codecov-io
Copy link

codecov-io commented Aug 21, 2018

Codecov Report

Merging #4754 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4754   +/-   ##
=======================================
  Coverage   20.69%   20.69%           
=======================================
  Files         167      167           
  Lines       32389    32389           
=======================================
  Hits         6702     6702           
  Misses      24706    24706           
  Partials      981      981

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 179123d...97fbb97. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 21, 2018
@bkcsoft bkcsoft added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Aug 21, 2018
@lafriks lafriks added the type/enhancement An improvement of existing functionality label Aug 21, 2018
@lafriks lafriks added this to the 1.6.0 milestone Aug 21, 2018
Copy link
Member

@daviian daviian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the button should be added to the "New File" and "Upload File" button group instead.

@bkcsoft bkcsoft added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 22, 2018
@decke
Copy link
Contributor Author

decke commented Aug 22, 2018

I will try to move the history button to the file actions and update the pull request.

@lunny
Copy link
Member

lunny commented Aug 29, 2018

@decke any update?

@decke
Copy link
Contributor Author

decke commented Aug 29, 2018

I'm currently on a business trip and will provide an update when I am back. Sorry for the delay.

decke added 2 commits August 30, 2018 12:45
The feature to list commits from a subdirectory is already there but
so far the history link to it was missing. There is a History button
in the view_file.tmpl already so avoid showing two history buttons in
that case.

The GitHub webinterface has the same History button in the same place
so this makes gitea a little bit more compatible.

Signed-off-by: Bernhard Froehlich <decke@bluelife.at>
@decke
Copy link
Contributor Author

decke commented Aug 30, 2018

That squashing didn't seem to work out properly. I will close this PR and create a new one.

@decke decke closed this Aug 30, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants