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 number of repos for every filter type #2312

Closed
Morlinest opened this issue Aug 16, 2017 · 13 comments
Closed

Add number of repos for every filter type #2312

Morlinest opened this issue Aug 16, 2017 · 13 comments
Labels
type/enhancement An improvement of existing functionality type/proposal The new feature has not been accepted yet but needs to be discussed first.
Milestone

Comments

@Morlinest
Copy link
Member

Morlinest commented Aug 16, 2017

Hi, what do you think about little improvement of UI (introduced in #2285):
image

or with grey color

image

Any comment or idea is welcome.

@lunny lunny added the type/enhancement An improvement of existing functionality label Aug 16, 2017
@lunny lunny added this to the 1.3.0 milestone Aug 16, 2017
@lafriks
Copy link
Member

lafriks commented Aug 16, 2017

Grey looks better. Also I think numbers should be lower like it is in my repositories count above.

@Morlinest
Copy link
Member Author

@lafriks Like this? Doesn't fit to screen well.

image

@Morlinest
Copy link
Member Author

I was looking at github dashboard. They have filters too, but each filter do own search. Our filter only filters result of search "all" repositories. I can rework rest api and add type to query options.

@daviian
Copy link
Member

daviian commented Aug 16, 2017

In my opinion numbers are too much. Why not just greyish the filter which does not have any results, don't disable the button, only greyish to give a visual hint?

@lafriks
Copy link
Member

lafriks commented Aug 16, 2017

I also does not like how it looks... Numbers makes interface with too much information. I agree that filters on each tab does need to perform it's own query as otherwise if you could get wrong information about your repositories if you are hitting page size limit

@lafriks lafriks added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Aug 16, 2017
@Morlinest
Copy link
Member Author

Morlinest commented Aug 17, 2017

@daviian @lafriks I agree with you both. I'll try to change search of repositories first, each filter should do own search. Also I have some idea how to make it effective (and after testing on github I found that they had same idea :D)

Edit: Not exactly the same idea (also some bugs there).

@Morlinest
Copy link
Member Author

Morlinest commented Aug 18, 2017

I have some updates. First, with #2326 it's possible to do search for each filter, it can looks like this:

  1. Without "counter" (same as now, but with custom search)
    2017-08-18_13-29-12

  2. Without "counter", with additional loader
    2017-08-18_13-26-39

  3. Can be same as 2 or 3, but with count of repositories for actual filter and searchQuery
    image
    image
    image

"Show more repositories" text is shown when actual search hits limit.

What do you think guys?

@lafriks
Copy link
Member

lafriks commented Aug 18, 2017

that looks good 👍

@Morlinest
Copy link
Member Author

@lafriks Which one looks best?

  1. With or without additional loader?
  2. With or without number of repositories?

@lafriks
Copy link
Member

lafriks commented Aug 18, 2017

@Morlinest with loader & with total repository count

@daviian
Copy link
Member

daviian commented Aug 20, 2017

@Morlinest One small approvement would be to show a little spinner (or something else) while searching instead of a zero count ;-)

@lafriks
Copy link
Member

lafriks commented Aug 20, 2017

No more spinners, two are already enough :) Just show count label only when data is loaded

@Morlinest
Copy link
Member Author

Closed by #2343

@lunny lunny modified the milestones: 1.x.x, 1.3.0 Nov 5, 2017
@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
type/enhancement An improvement of existing functionality type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

No branches or pull requests

4 participants