Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

docs: New Sponsor Popup#22393

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
Haviles04 wants to merge9 commits intomaster
base:master
Choose a base branch
Loading
fromhaviles04/support-snackbar
Draft

Conversation

@Haviles04
Copy link
Contributor

New sponsorship popup

@Haviles04Haviles04 requested review froma team andjohnleiderDecember 8, 2025 17:36
@KaelWDKaelWD changed the titlefeat: New Sponsor Popupdocs: New Sponsor PopupDec 9, 2025
@KaelWD
Copy link
Member

KaelWD commentedDec 9, 2025•
edited
Loading

Popups are yucky 🤢
We already have these ad spots all over the place
image

@J-Sek
Copy link
Contributor

Minor UI improvements + reordered links.

I don't mind experimenting with new placement. Those "ads via vuetify" do not show up on Brave.

Previewimage

@Haviles04
Copy link
ContributorAuthor

The idea was to mimic the lil popup that coolify has

@johnleider
Copy link
Member

Popups are yucky 🤢 We already have these ad spots all over the placeimage

Most of them are adblocked though.

@KaelWD
Copy link
Member

uBlock doesn't block them, only 5% of our traffic uses brave. What makes you think brave won't block this too?

@KaelWD
Copy link
Member

KaelWD commentedDec 12, 2025•
edited
Loading

If you're going to add this anyway the dismiss button needs to be remembered, and it probably shouldn't appear until either navigation or focus loss.

@johnleiderjohnleider marked this pull request as draftDecember 12, 2025 15:16
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@johnleiderjohnleiderAwaiting requested review from johnleider

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Haviles04@KaelWD@J-Sek@johnleider
[8]ページ先頭
©2009-2025 Movatter.jp