Skip to content

Conversation

Copy link
Member

@fregante fregante left a comment

Choose a reason for hiding this comment

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

A bit hesitant to add this feature, but since it's so simple let's just do it.

Let's also make the name more generic just in case we want to extend it

readme.md Outdated
- [](# "same-branch-author-commits") [Preserves current branch and path when viewing all commits by an author.](https://user-images.githubusercontent.com/44045911/148764372-ee443213-e61a-4227-9219-0ee54ed832e8.png)
- [](# "easy-toggle-commit-messages") [Enables toggling commit messages by clicking on the commit box.](https://user-images.githubusercontent.com/1402241/152121837-ca13bf8a-9b7f-4517-8e8d-b58bb135523b.gif)
- [](# "link-to-compare-diff") [Linkifies the "X files changed" text on compare pages to allow jumping to the diff.](https://user-images.githubusercontent.com/46634000/157072587-0335357a-18c7-44c4-ae6e-237080fb36b4.png)
- [](# "checks-modal") [Closes checks list when clicking outside of modal.]()
Copy link
Member

Choose a reason for hiding this comment

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

No screenshot needed

Copy link
Member

Choose a reason for hiding this comment

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

I meant the screenshot is not needed, but the mention in the readme yes

- [](# "checks-modal") Closes checks list when clicking outside of modal.

@fregante fregante changed the title Close checks list when clicking outside of modal click-outside-modal Dec 17, 2023
@fregante fregante changed the title click-outside-modal click-outside-modal - New feature Dec 17, 2023
@fregante

This comment was marked as off-topic.

@Katsute Katsute marked this pull request as ready for review December 17, 2023 05:03
@Katsute
Copy link
Contributor Author

Katsute commented Dec 17, 2023

^ squash followed by force push

@fregante

This comment was marked as resolved.

@fregante
Copy link
Member

I did it, thank you!

@fregante
Copy link
Member

You can run npm run fix to update the snapshots by the way

@fregante fregante self-assigned this Dec 17, 2023
@fregante fregante enabled auto-merge (squash) December 17, 2023 07:07
@fregante fregante merged commit 7339ae8 into refined-github:main Dec 17, 2023
@Katsute Katsute deleted the checks-modal branch December 17, 2023 17:48
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.

Close checks list when clicking outside of modal

2 participants