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

Remove Milestones filter if there are no milestones #421

Closed
hkdobrev opened this issue May 13, 2017 · 7 comments · Fixed by #2011
Closed

Remove Milestones filter if there are no milestones #421

hkdobrev opened this issue May 13, 2017 · 7 comments · Fixed by #2011

Comments

@hkdobrev
Copy link
Contributor

image

When there are no milestones the filter in issues and PRs listing page in a repo still have a filter for milestones which is not useful at all. There's just one option which just shows all issues/PRs with no milestones which are basically every issue/PR.

I think this would reduce clutter for projects not using milestones and not change anything if you're using milestones.

@sindresorhus
Copy link
Member

👍

@busches
Copy link
Member

busches commented May 23, 2017

I took a stab at this, but the milestone list isn't loaded until you attempt to filter on it. Is there another way we can determine if they're using milestones outside of what's in the list and if not, do we want to call the list all the time just to remove it?

busches@920554d

@sindresorhus
Copy link
Member

@busches Use MutationObserver.

@busches
Copy link
Member

busches commented May 24, 2017

@sindresorhus that seems clunky though. I'd click on Milestones and then it'd load the list, it has no milestones, and we'd immediately remove it?

@sindresorhus
Copy link
Member

Ok, I misunderstood how it worked.

@fregante
Copy link
Member

This requires a fetch to the API to check if a project has any Milestones. Is it worth it?

@yakov116
Copy link
Member

yakov116 commented Nov 15, 2018

@bfred-it I think only if its not your not part of the repo because
image
Thought were talking about assigning a milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

5 participants