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

proposal: split org Propfile README to a new tab overview #31373

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

a1012112796
Copy link
Member

like user profile, add a new overviw tab to show profile READEME when it is exist.

replace #31349 (another solution option)

example view:
屏幕截图 2024-06-14 094116
屏幕截图 2024-06-14 094101

like user profile, add a new overviw tab
to show profile READEME when it is exist.

Signed-off-by: a1012112796 <[email protected]>
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jun 14, 2024
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 14, 2024
@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files labels Jun 14, 2024
@a1012112796 a1012112796 added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Jun 14, 2024
@a1012112796 a1012112796 changed the title proposal: split Propfile README to a new tab overview proposal: split org Propfile README to a new tab overview Jun 14, 2024
@lunny
Copy link
Member

lunny commented Jun 14, 2024

Looks like Github's default page is repositories list.

@silverwind
Copy link
Member

Github renders:

  • Overview tab: Readme + Repos
  • Repos tab: Repos

I suggest we do the same, which should make everyone happy.

@silverwind
Copy link
Member

Looks like Github's default page is repositories list.

No, if I open https://github.com/go-gitea, it goes to the the overview tab.

@a1012112796
Copy link
Member Author

Github renders:

  • Overview tab: Readme + Repos
  • Repos tab: Repos

I suggest we do the same, which should make everyone happy.

has add it 990584f

{{svg "octicon-info"}} {{ctx.Locale.Tr "user.overview"}}
</a>
{{end}}
<a class="{{if .PageIsViewRepositories}}active {{end}}item" href="{{$.Org.HomeLink}}{{if .HasProfileReadme}}?view_repositorys=true{{end}}">
Copy link
Member

Choose a reason for hiding this comment

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

Can you make this route /<orgname>/repositories please? Query string is ugly and your view_repositorys is actually a typo.

@pull-request-size pull-request-size bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants