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 Pagination to Releases-page (and de-duplicate pagination templates) #73

Merged
merged 2 commits into from
Nov 8, 2016

Conversation

bkcsoft
Copy link
Member

@bkcsoft bkcsoft commented Nov 4, 2016

See #68

@bkcsoft bkcsoft added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels Nov 4, 2016
@codecov-io
Copy link

codecov-io commented Nov 4, 2016

Current coverage is 2.18% (diff: 0.00%)

Merging #73 into master will decrease coverage by <.01%

@@            master       #73   diff @@
========================================
  Files           31        31          
  Lines         7508      7510     +2   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           164       164          
- Misses        7327      7329     +2   
  Partials        17        17          

Powered by Codecov. Last update 03902bb...129f646

@@ -75,6 +75,7 @@
</li>
{{end}}
</ul>
{{template "admin/base/page" .}}
Copy link
Member

Choose a reason for hiding this comment

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

An admin pager in the frontend? ;)

Copy link
Member

Choose a reason for hiding this comment

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

We can copy admin/base/page to base/page and fix here since user's pager maybe not the same style with admin pager.

Copy link
Member Author

Choose a reason for hiding this comment

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

A agree that it is in the wrong place, but it does the right thing 😄

IMO it should be moved to base/pagination.tmpl and everything should use that. (Pagination should not require customization...)

@tboerger tboerger added this to the 1.0.0 milestone Nov 4, 2016
@bkcsoft bkcsoft changed the title Add Pagination to Releases-page [WIP] Add Pagination to Releases-page Nov 5, 2016
@xinity xinity added the pr/wip This PR is not ready for review label Nov 5, 2016
@strk strk modified the milestones: 1.1.0, 1.0.0 Nov 6, 2016
@bkcsoft
Copy link
Member Author

bkcsoft commented Nov 7, 2016

@tboerger @lunny better now? :)

@bkcsoft bkcsoft changed the title [WIP] Add Pagination to Releases-page [WIP] Add Pagination to Releases-page (and de-duplicate pagination templates) Nov 7, 2016
@codecov-io
Copy link

codecov-io commented Nov 7, 2016

Current coverage is 2.24% (diff: 0.00%)

Merging #73 into master will decrease coverage by <.01%

@@            master       #73   diff @@
========================================
  Files           32        32          
  Lines         7521      7523     +2   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           169       169          
- Misses        7335      7337     +2   
  Partials        17        17          

Powered by Codecov. Last update be5607e...562f9b6

@tboerger
Copy link
Member

tboerger commented Nov 7, 2016

LGTM

@bkcsoft bkcsoft changed the title [WIP] Add Pagination to Releases-page (and de-duplicate pagination templates) Add Pagination to Releases-page (and de-duplicate pagination templates) Nov 7, 2016
@bkcsoft bkcsoft removed the pr/wip This PR is not ready for review label Nov 7, 2016
@strk
Copy link
Member

strk commented Nov 7, 2016

LGTM (but didn't test visually, just trusting @bkcsoft )

@lunny lunny merged commit 5667d4d into go-gitea:master Nov 8, 2016
@lunny lunny modified the milestones: 1.0.0, 1.1.0 Nov 8, 2016
@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 29, 2016
lunny pushed a commit to lunny/gitea that referenced this pull request Feb 7, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 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. topic/ui Change the appearance of the Gitea UI type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants