Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Drop cycle-lists-with-keyboard-shortcuts feature - #3777

Merged
fregante merged 3 commits into
masterfrom
drop-cycle
Nov 29, 2020
Merged

Drop cycle-lists-with-keyboard-shortcuts feature#3777
fregante merged 3 commits into
masterfrom
drop-cycle

Conversation

@yakov116

@yakov116 yakov116 commented Nov 28, 2020

Copy link
Copy Markdown
Member
  1. LINKED ISSUES:
    Closes Drop cycle-lists-with-keyboard-shortcuts  #3776

  2. TEST URLS:
    None

  3. SCREENSHOT:
    None

@yakov116 yakov116 changed the title Drop cycle-lists-with-keyboard-shortcuts Drop cycle-lists-with-keyboard-shortcuts feature Nov 28, 2020
Comment thread readme.md Outdated
- [Blog post](https://github.com/sindresorhus/refined-github/pull/3015): [Makes it easier to tell apart commits added to the current PR versus plain commits that reference the PR.](https://user-images.githubusercontent.com/1402241/64478939-398b0a80-d1da-11e9-8c6a-bb98668cb78c.gif)
- [Blog post](https://github.blog/changelog/2020-06-23-design-updates-to-repositories-and-github-ui/): [Widens the discussion search box.](https://user-images.githubusercontent.com/1402241/55069759-bceaf080-50bf-11e9-84d0-7707de2eb9e9.png)

- [Blog post](https://github.com/sindresorhus/refined-github/issues/3776): [Allows the <kbd>↑</kbd> and <kbd>↓</kbd> keys to cycle "popover lists" (labels, milestones, etc).](https://user-images.githubusercontent.com/37769974/59158786-6fd2c400-8add-11e9-9db1-db80186fa6ea.gif)

@fregante fregante Nov 29, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

“Blog post” is starting to not make sense. Either we drop the link or we change the wording. It’s not particularly useful... I mean the whole list isn’t super useful either.

What if we replace it with a PR search link that points to every PR with “drop” in the title? We’ll also have to update some old PRs that don’t use the word.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Agree. This PR or a lint?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I was thinking the same. I was looking at the changes and when I say "blog post" I was confused.
What about simply using a label like adopted by github? So what you could use the link:
https://github.com/sindresorhus/refined-github/pulls?q=is%3Apr+is%3Aclosed+label%3A%22adopted+by+github%22

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about simply using a label

We could use a more generic dropped feature label, but we don't really need another label since the PR title should include that keyword already.

Agree. This PR or a lint?

This PR can just avoid making changes to that section. In a lint you could drop the list and if you have time maybe also track down any PRs that don't have the "Drop" keyword in the title

@fregante
fregante merged commit dd783c9 into master Nov 29, 2020
@fregante
fregante deleted the drop-cycle branch November 29, 2020 23:48
@yakov116 yakov116 added meta Related to Refined GitHub itself implemented by github and removed meta Related to Refined GitHub itself labels Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Drop cycle-lists-with-keyboard-shortcuts

3 participants