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

feature: support auto-hide comments when page navigated #40

Merged
merged 2 commits into from
Mar 26, 2018

Conversation

1natsu172
Copy link
Collaborator

@1natsu172 1natsu172 commented Mar 9, 2018

Issue: #19

I implemented the feature discussed in the issue.
#21 was submitting from the fork repository, so I resubmitted this PR.

Please review when have time. 🙏

When navigate with the comments open, it auto-hide the comment.
code formatted prettier.
Copy link
Owner

@dideler dideler left a comment

Choose a reason for hiding this comment

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

Thanks! Code looks good. I'll test it in the browser when I get home.

Copy link
Owner

@dideler dideler left a comment

Choose a reason for hiding this comment

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

Tested and works great! 🚀

@1natsu172
Copy link
Collaborator Author

@dideler Thanks for approval.

This is a confirmation for a moment. I think this new feature is not backward compatible (UI doesn't change but UX changes). I'm aware that it is v4.0.0.
If there is no problem merge and release work. Maybe around tomorrow.

@dideler
Copy link
Owner

dideler commented Mar 25, 2018

Hey @1natsu172, I was away for another holiday (this time in Wales) - thanks for your patience.

Would you like to release this feature? Since you built it, you can have the pleasure of shipping it.

@1natsu172
Copy link
Collaborator Author

1natsu172 commented Mar 25, 2018

Welcome back @dideler .

Yes, I'll release this feature. Maybe I can take time tomorrow or the day after tomorrow. (Since I don't have time today, I only reply).

@1natsu172 1natsu172 merged commit 49308a2 into master Mar 26, 2018
@1natsu172 1natsu172 deleted the #19/dev/support-pageJumpAction branch March 26, 2018 07:11
@1natsu172
Copy link
Collaborator Author

1natsu172 commented Mar 26, 2018

@dideler, Now I released it as v4.0.0! 🚀

I'm very grateful for the discussion at issue #19 and the your user's opinion survey!

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

Successfully merging this pull request may close these issues.

2 participants