Skip to content

Rename more-dropdown to more-dropdown-links#4584

Merged
yakov116 merged 4 commits intomainfrom
more-dropdown-links
Jul 19, 2021
Merged

Rename more-dropdown to more-dropdown-links#4584
yakov116 merged 4 commits intomainfrom
more-dropdown-links

Conversation

@yakov116
Copy link
Member

@yakov116 yakov116 commented Jul 19, 2021

Migration will happen in #4568, after the feature is renamed.

@fregante
Copy link
Member

Unchecked but I assume it matches the previous code. Merge at will

readme.md Outdated
@@ -134,6 +134,7 @@ Thanks for contributing! 🦋🙌
- [](# "hide-watch-and-fork-count") [Hides watcher counter and on smaller screens the fork counter too.](https://user-images.githubusercontent.com/1402241/53681077-f3328b80-3d1e-11e9-9e29-2cb017141769.png)
- [](# "sort-milestones-by-closest-due-date") Changes the default sort order of milestones `Closest due date`.
- [](# "more-dropdown") [Adds useful links to the repository navigation dropdown and moves the "Security" and "Insights" tabs to it as well.](https://user-images.githubusercontent.com/16872793/124681432-856e6f80-de96-11eb-89c9-6d78e8ae4329.png)
Copy link
Member

Choose a reason for hiding this comment

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

This line should be dropped, right?

}

async function init(): Promise<void> {
// The user may have disabled `more-dropdown` so un-hide it
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// The user may have disabled `more-dropdown` so un-hide it
// The user may have disabled `more-dropdown-links` so un-hide it

@yakov116 yakov116 merged commit f56ac11 into main Jul 19, 2021
@yakov116 yakov116 deleted the more-dropdown-links branch July 19, 2021 10:19
@kdcloverkid

This comment has been minimized.

@fregante fregante mentioned this pull request Jan 10, 2026
9 tasks
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.

4 participants