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

netiquette - Hide from preview tab #7613

Merged
merged 1 commit into from
Jul 28, 2024
Merged

netiquette - Hide from preview tab #7613

merged 1 commit into from
Jul 28, 2024

Conversation

kidonng
Copy link
Member

@kidonng kidonng commented Jul 28, 2024

Fix #7612

Test URLs

Screenshot

draft-pr-notice

Screen.Recording.2024-07-29.at.3.13.11.AM.mov

netiquette

Screen.Recording.2024-07-29.at.3.14.30.AM.mov

@kidonng kidonng added the bug label Jul 28, 2024
@fregante
Copy link
Member

Should we just merge the two features? 🤔

@kidonng
Copy link
Member Author

kidonng commented Jul 28, 2024

That was my first thought when I made #7514 but I put it off so we can have something usable first. I have no issue if you agree to merge them.

@fregante
Copy link
Member

The last rename was just a week ago #7570 and the feature wasn't mentioned anywhere else I think. Perfect time to roll it into netiquette and treat it as another rename.

@fregante

This comment has been minimized.

@fregante fregante changed the title Hide comment banners in preview tab netiquette - Hide banners in preview tab Jul 28, 2024
@fregante fregante merged commit f31bf95 into main Jul 28, 2024
12 checks passed
@fregante fregante deleted the fix/comment-banner branch July 28, 2024 19:36
@fregante
Copy link
Member

I'll merge them later together with the screenshot 🙏

I wanted to merge this as a bugfix first so it appears in the release notes.

@fregante fregante changed the title netiquette - Hide banners in preview tab netiquette - Hide from preview tab Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

netiquette appears in preview tab
2 participants