gitea icon indicating copy to clipboard operation
gitea copied to clipboard

Add user/organization code search

Open lafriks opened this issue 3 years ago • 3 comments

Fixes #19925

Screenshots: attels

lafriks avatar Jun 15 '22 15:06 lafriks

It might be a good Idea to also add search for Issue/PRs and other things.

JakobDev avatar Aug 23 '22 13:08 JakobDev

That is out of scope for this PR

lafriks avatar Sep 01 '22 11:09 lafriks

@delvh please review again

lafriks avatar Sep 16 '22 16:09 lafriks

@delvh done, please review

lafriks avatar Sep 27 '22 17:09 lafriks

Absolute URL like HTMLURL should not be used.

  • https://github.com/go-gitea/gitea/issues/19345

wxiaoguang avatar Oct 09 '22 11:10 wxiaoguang

@wxiaoguang what would be the alternative?

delvh avatar Oct 09 '22 13:10 delvh

@wxiaoguang what would be the alternative?

Always use relative links(like "/AppSubUrl/org/repo/....") on Web UI

wxiaoguang avatar Oct 09 '22 13:10 wxiaoguang

Oh, and another thing I've noticed (doesn't need to be fixed in this PR): Why exactly do we make another request when selecting a type? Couldn't we also do that in the frontend?

delvh avatar Oct 09 '22 15:10 delvh

@delvh these changes should be done in other PR as otherwise look and naming will be inconsistent with global and repository code search pages!

lafriks avatar Oct 10 '22 09:10 lafriks

There are already too many refactorings that now touches code outside of scope of this PR

lafriks avatar Oct 10 '22 09:10 lafriks

Could we avoid squash or rebase + force push when review starts?

It's not clear what has changed since last review, the review should start from beginning again .........

wxiaoguang avatar Oct 10 '22 15:10 wxiaoguang

@delvh these changes should be done in other PR as otherwise look and naming will be inconsistent with global and repository code search pages!

I accept not renaming Match to Exact. But everything else is absolutely needed. I strongly disagree that displaying the tooltips is out of scope. Or do you mean because the tooltips aren't present in the other search, they shouldn't be added??? In that case I disagree even more. Simply because this wassn't done well previously doesn't mean we need to do it badly now.

delvh avatar Oct 10 '22 16:10 delvh

Added tooltips and refactored code in all code search places to be consistent

lafriks avatar Oct 10 '22 20:10 lafriks

Make lgtm work

zeripath avatar Oct 10 '22 23:10 zeripath