-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Implementation of Folder Jumping #511
Conversation
LGTM Very good. After go-gitea/git#23 being merged, please do update vendor e push another commit:
|
Great feature, but I personally prefer the Github approach (greying out empty folders). Maybe you could add that? |
@@ -1272,6 +1272,9 @@ footer .ui.language .menu { | |||
.repository.file.list #repo-files-table tr:hover { | |||
background-color: #ffffEE; | |||
} | |||
.repository.file.list #repo-files-table .jumpable-path { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Css files get generated by less
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rule added to the less file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great 👍
{{else}} | ||
{{index $subJumpablePath 0}} | ||
{{end}} | ||
</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong indentation?
@sriio You feel done with this? in that case |
LGTM |
vendor updated and squash done |
Simple implementation of Folder Jumping
Comes with this PR